Wed, 16 Jul 2008 10:08:57 -0700
6707044: uncommon_trap of ifnull bytecode leaves garbage on expression stack
Summary: Remove call to repush_if_args()
Reviewed-by: kvn, jrose
src/share/vm/opto/parse2.cpp | file | annotate | diff | comparison | revisions |
1.1 --- a/src/share/vm/opto/parse2.cpp Fri Jul 11 12:19:29 2008 -0700 1.2 +++ b/src/share/vm/opto/parse2.cpp Wed Jul 16 10:08:57 2008 -0700 1.3 @@ -875,6 +875,8 @@ 1.4 return prob < PROB_MIN; 1.5 } 1.6 1.7 +//-------------------------------repush_if_args-------------------------------- 1.8 +// Push arguments of an "if" bytecode back onto the stack by adjusting _sp. 1.9 inline void Parse::repush_if_args() { 1.10 #ifndef PRODUCT 1.11 if (PrintOpto && WizardMode) { 1.12 @@ -906,7 +908,6 @@ 1.13 if (PrintOpto && Verbose) 1.14 tty->print_cr("Never-taken backedge stops compilation at bci %d",bci()); 1.15 #endif 1.16 - repush_if_args(); // to gather stats on loop 1.17 // We need to mark this branch as taken so that if we recompile we will 1.18 // see that it is possible. In the tiered system the interpreter doesn't 1.19 // do profiling and by the time we get to the lower tier from the interpreter