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

Igor Veresov igor.veresov at
Tue Oct 4 09:46:05 PDT 2011

 Thanks Vladimir and Tom! 


On Tuesday, October 4, 2011 at 9:06 AM, Tom Rodriguez wrote:

> Looks good.
> tom
> 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

-------------- next part --------------
An HTML attachment was scrubbed...

More information about the hotspot-compiler-dev mailing list