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

This reverts commit b63b4a28ed2cdbfae75fb5e550313ce3ccec1eae.

Reason for revert: Failures on CtsSimpleperfTestCases.record_cmd#check_trampoline_after_art_jni_methods b/300064491

Change-Id: Ia666ae211b1f2cf4548bb5844ec22c3f878c8cab

1 files changed, 1 insertions, 4 deletions

diff --git a/runtime/class_linker.cc b/runtime/class_linker.cc
index 953cc676dc..1edb694a1d 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_);