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

6707044: uncommon_trap of ifnull bytecode leaves garbage on expression stack

Browse source 
Remove call to repush_if_args()

Reviewed-by: kvn, jrose
This commit is contained in:
Chuck Rasbold 2008-07-16 10:08:57 -07:00
parent c5b951e990
commit a8fc1db8c1
1 changed files with 2 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

3
hotspot/src/share/vm/opto/parse2.cpp
View file

@ -875,6 +875,8 @@ bool Parse::seems_never_taken(float prob) {
return prob < PROB_MIN; 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() { inline void Parse::repush_if_args() {
#ifndef PRODUCT #ifndef PRODUCT
if (PrintOpto && WizardMode) { if (PrintOpto && WizardMode) {
@ -906,7 +908,6 @@ void Parse::do_ifnull(BoolTest::mask btest) {
if (PrintOpto && Verbose) if (PrintOpto && Verbose)
tty->print_cr("Never-taken backedge stops compilation at bci %d",bci()); tty->print_cr("Never-taken backedge stops compilation at bci %d",bci());
#endif #endif
repush_if_args(); // to gather stats on loop
// We need to mark this branch as taken so that if we recompile we will // 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 // 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 // do profiling and by the time we get to the lower tier from the interpreter