# HG changeset patch
# User coleenp
# Date 1426772847 14400
# Node ID d2b6dea68cb6724a6a37e286c84406e49313ee07
# Parent  6b22ac2d94cd45d110f3821353f81c314a1643b1
8005521: StressMethodComparator is not thread-safe
Summary: Remove the debug option. It isn't correct or tested.
Reviewed-by: gtriantafill, dholmes, sspitsyn

diff -r 6b22ac2d94cd -r d2b6dea68cb6 hotspot/src/share/vm/oops/instanceKlass.cpp
--- a/hotspot/src/share/vm/oops/instanceKlass.cpp	Wed Mar 18 17:45:47 2015 -0400
+++ b/hotspot/src/share/vm/oops/instanceKlass.cpp	Thu Mar 19 09:47:27 2015 -0400
@@ -716,23 +716,6 @@
 
     // Set up method entry points for compiler and interpreter    .
     m->link_method(m, CHECK);
-
-    // This is for JVMTI and unrelated to relocator but the last thing we do
-#ifdef ASSERT
-    if (StressMethodComparator) {
-      ResourceMark rm(THREAD);
-      static int nmc = 0;
-      for (int j = i; j >= 0 && j >= i-4; j--) {
-        if ((++nmc % 1000) == 0)  tty->print_cr("Have run MethodComparator %d times...", nmc);
-        bool z = MethodComparator::methods_EMCP(m(),
-                   methods()->at(j));
-        if (j == i && !z) {
-          tty->print("MethodComparator FAIL: "); m->print(); m->print_codes();
-          assert(z, "method must compare equal to itself");
-        }
-      }
-    }
-#endif //ASSERT
   }
 }
 
diff -r 6b22ac2d94cd -r d2b6dea68cb6 hotspot/src/share/vm/runtime/globals.hpp
--- a/hotspot/src/share/vm/runtime/globals.hpp	Wed Mar 18 17:45:47 2015 -0400
+++ b/hotspot/src/share/vm/runtime/globals.hpp	Thu Mar 19 09:47:27 2015 -0400
@@ -1342,9 +1342,6 @@
   product(intx, TraceRedefineClasses, 0,                                    \
           "Trace level for JVMTI RedefineClasses")                          \
                                                                             \
-  develop(bool, StressMethodComparator, false,                              \
-          "Run the MethodComparator on all loaded methods")                 \
-                                                                            \
   /* change to false by default sometime after Mustang */                   \
   product(bool, VerifyMergedCPBytecodes, true,                              \
           "Verify bytecodes after RedefineClasses constant pool merging")   \