RFR(S): 8247307: C2: Loop array fill stub routines are not called

Tobias Hartmann tobias.hartmann at oracle.com
Thu Jun 25 13:11:11 UTC 2020


Hi Pengfei,

On 24.06.20 06:21, Pengfei Li wrote:
> In the original code, it's supposed that no node in the loop can be used by nodes outside the loop in the array fill pattern. But that's not true for CountedLoopNode in strip-mined counted loops. So my change just skips this use. It's a good idea to bail out if use != polladr. I changed the if condition to an assertion and ran the jtreg cases I tested before. There's no new failure so I think there's no other use cases.

Right, an assert is better.

> Done and uploaded a new webrev: http://cr.openjdk.java.net/~pli/rfr/8247307/webrev.01/

Looks good.

Best regards,
Tobias


More information about the hotspot-compiler-dev mailing list