From 18fba4cee0da2a2ca3051791a9cbdf5fa6f552c9 Mon Sep 17 00:00:00 2001
From: Mythri Alle <mythria@google.com>
Date: Wed, 27 Oct 2021 10:00:55 +0000
Subject: Remove unused code related to method entry / exit events

This Cl:
- Drops this_object and dex_pc arguments for MethodExitEvent
  which aren't used in any of the listeners.
- Removes the code to maintain dex_pcs_ which were earlier used
  by MethodEnteredEvent but are no longer used.

Test: test/tes.py
Change-Id: I0345598897004b7b6b9f26bd8940c88b5bb798f7
---
 runtime/interpreter/interpreter_switch_impl-inl.h | 20 ++++----------------
 1 file changed, 4 insertions(+), 16 deletions(-)

(limited to 'runtime/interpreter/interpreter_switch_impl-inl.h')

diff --git a/runtime/interpreter/interpreter_switch_impl-inl.h b/runtime/interpreter/interpreter_switch_impl-inl.h
index 8e16e04bfa..42f7fc749b 100644
--- a/runtime/interpreter/interpreter_switch_impl-inl.h
+++ b/runtime/interpreter/interpreter_switch_impl-inl.h
@@ -64,12 +64,8 @@ class InstructionHandler {
       DCHECK(abort_exception != nullptr);
       DCHECK(abort_exception->GetClass()->DescriptorEquals(Transaction::kAbortExceptionDescriptor));
       Self()->ClearException();
-      PerformNonStandardReturn<kMonitorState>(Self(),
-                                              shadow_frame_,
-                                              ctx_->result,
-                                              Instrumentation(),
-                                              Accessor().InsSize(),
-                                              inst_->GetDexPc(Insns()));
+      PerformNonStandardReturn<kMonitorState>(
+          Self(), shadow_frame_, ctx_->result, Instrumentation(), Accessor().InsSize());
       Self()->SetException(abort_exception.Get());
       ExitInterpreterLoop();
       return false;
@@ -80,12 +76,8 @@ class InstructionHandler {
   HANDLER_ATTRIBUTES bool CheckForceReturn() {
     if (shadow_frame_.GetForcePopFrame()) {
       DCHECK(Runtime::Current()->AreNonStandardExitsEnabled());
-      PerformNonStandardReturn<kMonitorState>(Self(),
-                                              shadow_frame_,
-                                              ctx_->result,
-                                              Instrumentation(),
-                                              Accessor().InsSize(),
-                                              inst_->GetDexPc(Insns()));
+      PerformNonStandardReturn<kMonitorState>(
+          Self(), shadow_frame_, ctx_->result, Instrumentation(), Accessor().InsSize());
       ExitInterpreterLoop();
       return false;
     }
@@ -216,9 +208,7 @@ class InstructionHandler {
                  !SendMethodExitEvents(Self(),
                                        Instrumentation(),
                                        shadow_frame_,
-                                       shadow_frame_.GetThisObject(Accessor().InsSize()),
                                        shadow_frame_.GetMethod(),
-                                       inst_->GetDexPc(Insns()),
                                        result))) {
       DCHECK(Self()->IsExceptionPending());
       // Do not raise exception event if it is caused by other instrumentation event.
@@ -495,9 +485,7 @@ class InstructionHandler {
                  !SendMethodExitEvents(Self(),
                                        Instrumentation(),
                                        shadow_frame_,
-                                       shadow_frame_.GetThisObject(Accessor().InsSize()),
                                        shadow_frame_.GetMethod(),
-                                       inst_->GetDexPc(Insns()),
                                        h_result))) {
       DCHECK(Self()->IsExceptionPending());
       // Do not raise exception event if it is caused by other instrumentation event.
-- 
cgit v1.2.3-59-g8ed1b