archive/jdk
1
0
Fork 0
mirror of https://github.com/openjdk/jdk.git synced 2025-09-21 03:24:38 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

6385687: UseFastEmptyMethods/UseFastAccessorMethods considered harmful

Browse source 
Reviewed-by: kvn, jrose, phh
This commit is contained in:
Tom Rodriguez 2011-03-31 21:05:55 -07:00
parent f33b8cbb7a
commit 3ad32dbe57
2 changed files with 7 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

7
hotspot/src/share/vm/prims/jvmtiManageCapabilities.cpp
View file

@ -319,8 +319,11 @@ void JvmtiManageCapabilities::update() {
bool enter_all_methods = bool enter_all_methods =
interp_events || interp_events ||
avail.can_generate_breakpoint_events; avail.can_generate_breakpoint_events;
UseFastEmptyMethods = !enter_all_methods; if (enter_all_methods) {
UseFastAccessorMethods = !enter_all_methods; // Disable these when tracking the bytecodes
UseFastEmptyMethods = false;
UseFastAccessorMethods = false;
}
if (avail.can_generate_breakpoint_events) { if (avail.can_generate_breakpoint_events) {
RewriteFrequentPairs = false; RewriteFrequentPairs = false;

4
hotspot/src/share/vm/runtime/globals.hpp
View file

@ -2726,10 +2726,10 @@ class CommandLineFlags {
product(bool, UseLoopCounter, true, \ product(bool, UseLoopCounter, true, \
"Increment invocation counter on backward branch") \ "Increment invocation counter on backward branch") \
\ \
product(bool, UseFastEmptyMethods, true, \ product(bool, UseFastEmptyMethods, false, \
"Use fast method entry code for empty methods") \ "Use fast method entry code for empty methods") \
\ \
product(bool, UseFastAccessorMethods, true, \ product(bool, UseFastAccessorMethods, false, \
"Use fast method entry code for accessor methods") \ "Use fast method entry code for accessor methods") \
\ \
product_pd(bool, UseOnStackReplacement, \ product_pd(bool, UseOnStackReplacement, \