6385687: UseFastEmptyMethods/UseFastAccessorMethods considered harmful
authornever
Thu, 31 Mar 2011 21:05:55 -0700
changeset 9098 458f0d0809b9
parent 8887 1e9e60a225a6
child 9099 bdeb610d3cb1
6385687: UseFastEmptyMethods/UseFastAccessorMethods considered harmful Reviewed-by: kvn, jrose, phh
hotspot/src/share/vm/prims/jvmtiManageCapabilities.cpp
hotspot/src/share/vm/runtime/globals.hpp
--- a/hotspot/src/share/vm/prims/jvmtiManageCapabilities.cpp	Thu Mar 31 16:54:27 2011 -0700
+++ b/hotspot/src/share/vm/prims/jvmtiManageCapabilities.cpp	Thu Mar 31 21:05:55 2011 -0700
@@ -319,8 +319,11 @@
   bool enter_all_methods =
     interp_events ||
     avail.can_generate_breakpoint_events;
-  UseFastEmptyMethods = !enter_all_methods;
-  UseFastAccessorMethods = !enter_all_methods;
+  if (enter_all_methods) {
+    // Disable these when tracking the bytecodes
+    UseFastEmptyMethods = false;
+    UseFastAccessorMethods = false;
+  }
 
   if (avail.can_generate_breakpoint_events) {
     RewriteFrequentPairs = false;
--- a/hotspot/src/share/vm/runtime/globals.hpp	Thu Mar 31 16:54:27 2011 -0700
+++ b/hotspot/src/share/vm/runtime/globals.hpp	Thu Mar 31 21:05:55 2011 -0700
@@ -2726,10 +2726,10 @@
   product(bool, UseLoopCounter, true,                                       \
           "Increment invocation counter on backward branch")                \
                                                                             \
-  product(bool, UseFastEmptyMethods, true,                                  \
+  product(bool, UseFastEmptyMethods, false,                                 \
           "Use fast method entry code for empty methods")                   \
                                                                             \
-  product(bool, UseFastAccessorMethods, true,                               \
+  product(bool, UseFastAccessorMethods, false,                              \
           "Use fast method entry code for accessor methods")                \
                                                                             \
   product_pd(bool, UseOnStackReplacement,                                   \