RFR(XS): 8149543: range check CastII nodes should not be split through Phi

Roland Westrelin roland.westrelin at oracle.com
Fri Feb 12 12:22:35 UTC 2016


Thanks for looking at this, Vladimir.

> It would be nice if we could narrow this bailout only for loops. I mean when Phi's control is loop's head.
> If it is an ordinary merge region splitting through it may help to improve performance. What do you think?
> Could you run some performance tests to effects. If ordinary region cases are rare then you current fix is good enough.

What about this?
http://cr.openjdk.java.net/~roland/8149543/webrev.01/

Tobias’ performance runs when he added the range check CastII found no regressions so I suppose test runs where I tweak how much split thru phi we do for CastII nodes are unlikely to show anything either. What do you think?

Roland.


More information about the hotspot-compiler-dev mailing list