# HG changeset patch # User rasbold # Date 1216228137 25200 # Node ID 9b66e6287f4acad10b8d498d16077cf4703ec349 # Parent 4a4c365f777dd250f9ce1ad654de733f188e3174 6707044: uncommon_trap of ifnull bytecode leaves garbage on expression stack Summary: Remove call to repush_if_args() Reviewed-by: kvn, jrose diff -r 4a4c365f777d -r 9b66e6287f4a src/share/vm/opto/parse2.cpp --- a/src/share/vm/opto/parse2.cpp Fri Jul 11 12:19:29 2008 -0700 +++ b/src/share/vm/opto/parse2.cpp Wed Jul 16 10:08:57 2008 -0700 @@ -875,6 +875,8 @@ return prob < PROB_MIN; } +//-------------------------------repush_if_args-------------------------------- +// Push arguments of an "if" bytecode back onto the stack by adjusting _sp. inline void Parse::repush_if_args() { #ifndef PRODUCT if (PrintOpto && WizardMode) { @@ -906,7 +908,6 @@ if (PrintOpto && Verbose) tty->print_cr("Never-taken backedge stops compilation at bci %d",bci()); #endif - repush_if_args(); // to gather stats on loop // We need to mark this branch as taken so that if we recompile we will // see that it is possible. In the tiered system the interpreter doesn't // do profiling and by the time we get to the lower tier from the interpreter