From 38c20d4a694eea44a1bd3af32a6a99512c139358 Mon Sep 17 00:00:00 2001 From: Alex Light Date: Sat, 11 Feb 2017 00:38:35 +0000 Subject: Revert "Revert "Revert "Make class redefinition work with native methods on stack.""" This reverts commit 4493f92dfc70e01d95ad57bc4c0c38f59a616f70. Reason for revert: Fails interpreter-ac and relocate-npatchoat Change-Id: I15eb4964c9101fa04e34b52acf9ffe25e6d4e60d Test: revert --- runtime/stack.cc | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) (limited to 'runtime/stack.cc') diff --git a/runtime/stack.cc b/runtime/stack.cc index 96fc664584..d7ba1d75d8 100644 --- a/runtime/stack.cc +++ b/runtime/stack.cc @@ -874,11 +874,9 @@ void StackVisitor::WalkStack(bool include_transitions) { CHECK_EQ(GetMethod(), callee) << "Expected: " << ArtMethod::PrettyMethod(callee) << " Found: " << ArtMethod::PrettyMethod(GetMethod()); } else { - // Instrumentation generally doesn't distinguish between a method's obsolete and - // non-obsolete version. - CHECK_EQ(instrumentation_frame.method_, GetMethod()->GetNonObsoleteMethod()) + CHECK_EQ(instrumentation_frame.method_, GetMethod()) << "Expected: " << ArtMethod::PrettyMethod(instrumentation_frame.method_) - << " Found: " << ArtMethod::PrettyMethod(GetMethod()->GetNonObsoleteMethod()); + << " Found: " << ArtMethod::PrettyMethod(GetMethod()); } if (num_frames_ != 0) { // Check agreement of frame Ids only if num_frames_ is computed to avoid infinite @@ -905,7 +903,7 @@ void StackVisitor::WalkStack(bool include_transitions) { << " native=" << method->IsNative() << std::noboolalpha << " entrypoints=" << method->GetEntryPointFromQuickCompiledCode() - << "," << (method->IsNative() ? method->GetEntryPointFromJni() : nullptr) + << "," << method->GetEntryPointFromJni() << " next=" << *cur_quick_frame_; } -- cgit v1.2.3