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