6707044: uncommon_trap of ifnull bytecode leaves garbage on expression stack
Summary: Remove call to repush_if_args()
Reviewed-by: kvn, jrose
--- a/hotspot/src/share/vm/opto/parse2.cpp Fri Jul 11 12:19:29 2008 -0700
+++ b/hotspot/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