Revert "Don't use shared hotness counters for debuggable runtimes"

This reverts commit 8f13c9d50fdc8bc03d0091eefd19dda7d2234fcc.

Reason for revert: Breaks CtsSimpleperfTestCases record_cmd#check_trampoline_after_art_jni_methods
b/296416302

Bug: 296416302
Change-Id: I9079788e289407fdd1b417874a0abe4d2c5e1c9a

1 files changed, 1 insertions, 4 deletions

diff --git a/runtime/class_linker.cc b/runtime/class_linker.cc
index fb1b4651ce..a476ed67cc 100644
--- a/runtime/class_linker.cc
+++ b/runtime/class_linker.cc
@@ -2236,14 +2236,11 @@ bool ClassLinker::AddImageSpace(gc::space::ImageSpace* space,
     // If we are profiling the boot classpath, we disable the shared memory
     // optimization to make sure boot classpath methods all get properly
     // profiled.
-    // For debuggable runtimes we don't use AOT code, so don't use shared memory
-    // optimization so the methods can be JITed better.
     //
     // We need to disable the flag before doing ResetCounter below, as counters
     // of shared memory method always hold the "hot" value.
     if (!runtime->IsZygote() ||
-        runtime->GetJITOptions()->GetProfileSaverOptions().GetProfileBootClassPath() ||
-        runtime->IsJavaDebuggable()) {
+        runtime->GetJITOptions()->GetProfileSaverOptions().GetProfileBootClassPath()) {
       header.VisitPackedArtMethods([&](ArtMethod& method) REQUIRES_SHARED(Locks::mutator_lock_) {
         method.ClearMemorySharedMethod();
       }, space->Begin(), image_pointer_size_);