review(XS): 6591247: C2 cleans up the merge point too early during SplitIf.

Tom Rodriguez tom.rodriguez at
Thu Aug 25 19:57:11 PDT 2011

Looks good.


On Aug 25, 2011, at 5:10 PM, Igor Veresov wrote:

> The problem here is that during split-if we remove the region's self reference too early while processing its users, which can make get_ctrl_no_update() return the wrong answer. 
> I wasn't able to reproduce the problem, but it seems to be possible for it to occur if the region points to something else but phi and the self reference is deleted too early. 
> The solution is to remove the self reference last.
> Webrev:
> Testing: specjvm98, CTW
> Thanks,
> igor

More information about the hotspot-compiler-dev mailing list