From 1d1d25eea72cf22aed802352a82588d97403f7b6 Mon Sep 17 00:00:00 2001 From: Mythri Alle Date: Mon, 4 Jul 2022 12:00:14 +0000 Subject: Revert "Reland "Don't use instrumentation stubs for native methods in debuggable"" This reverts commit 6fb0acc14459a856c35b642e3368aff853259260. Reason for revert: Breaks android.jvmti.cts.JvmtiHostTest https://buganizer.corp.google.com/issues/237991413 Change-Id: I00fb58080693ddebc03c7b62ea67c91150ef7a21 --- runtime/quick_exception_handler.cc | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) (limited to 'runtime/quick_exception_handler.cc') diff --git a/runtime/quick_exception_handler.cc b/runtime/quick_exception_handler.cc index 40a1c16905..8adc3b3e49 100644 --- a/runtime/quick_exception_handler.cc +++ b/runtime/quick_exception_handler.cc @@ -399,10 +399,9 @@ class DeoptimizeStackVisitor final : public StackVisitor { return true; } else if (method->IsNative()) { // If we return from JNI with a pending exception and want to deoptimize, we need to skip - // the native method. The top method is a runtime method, the native method comes next. - // We also deoptimize due to method instrumentation reasons from method entry / exit - // callbacks. In these cases native method is at the top of stack. - CHECK((GetFrameDepth() == 1U) || (GetFrameDepth() == 0U)); + // the native method. + // The top method is a runtime method, the native method comes next. + CHECK_EQ(GetFrameDepth(), 1U); callee_method_ = method; return true; } else if (!single_frame_deopt_ && -- cgit v1.2.3-59-g8ed1b