RFR(M): 8223051: support loops with long (64b) trip counts

John Rose john.r.rose at oracle.com
Tue May 26 21:14:34 UTC 2020


On May 6, 2020, at 2:41 AM, Roland Westrelin <rwestrel at redhat.com> wrote:
> 
>> So can we arrange to run it more than once, by setting the inner trip
>> count to be smaller?  I’m afraid the optimizer could detect a one-trip loop
>> and take it apart (in a later pass), and then the goal of the stress test
>> won’t be achieved.
> 
> Sure. Here is a new patch that should take all of your comments into
> account:
> 
> http://cr.openjdk.java.net/~roland/8223051/webrev.01/
> 
> I split the change in 2. The counted loop refactoring code is out for
> review under 8244504 (I just sent it out for review). Your review of
> that other one is very much welcome.

So, I’m very happy with the refactoring 8244504 work.

Maybe someone else will have comments 0n that, but assuming
that is more or less unchanged, this change set for 8223051 on
top still looks good, taking into account renaming of min/max
factories.

After you post an updated webrev.02 the review should be easy.
Tobias might wish to run some regression tests on the final changes.

— John


More information about the hotspot-compiler-dev mailing list