review(S): 7097679: Tiered: events with bad bci to Gotos reduced from Ifs

Tom Rodriguez tom.rodriguez at
Tue Oct 4 09:06:19 PDT 2011

Looks good.


On Oct 3, 2011, at 10:48 PM, Igor Veresov wrote:

> In case with back branches Runtime1::counter_overflow() routine expects to receive a bci of either a Goto of If instruction that generated
> the event. When emitting the counter overflow stub for the Goto we take the bci from the jvm state. However it is not correct for the Gotos 
> that were produced as a result of the reduction of Ifs because in this case the state will have a bci that would possibly point to a Cmp preceding the If instead.
> The solution is to always store a proper bci for Goto and use it instead of the one in the jvm state.
> Webrev:
> Thanks!
> igor

More information about the hotspot-compiler-dev mailing list