6707044: uncommon_trap of ifnull bytecode leaves garbage on expression stack
authorrasbold
Wed, 16 Jul 2008 10:08:57 -0700
changeset 954 740f5b9a923e
parent 953 202a1f972a92
child 955 723d7b577fba
6707044: uncommon_trap of ifnull bytecode leaves garbage on expression stack Summary: Remove call to repush_if_args() Reviewed-by: kvn, jrose
hotspot/src/share/vm/opto/parse2.cpp
--- 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