Integrated: 8311087: PhiNode::wait_for_region_igvn should break early

Johan Sjölen jsjolen at openjdk.org
Thu Jul 6 12:30:07 UTC 2023


On Thu, 29 Jun 2023 10:44:45 GMT, Johan Sjölen <jsjolen at openjdk.org> wrote:

> Hi, please consider this PR.
> 
> Instead of continuing the loop we break after setting `delay = true`. I also flattened out the indentation by transforming `A && B` into `!A || !B` and `continue`ing if so. This makes the code a bit longer, but clearer imho.
> 
> I've run `TestCastIIAfterUnrollingInOuterLoop`, which is the test that was added along with this method. I'm also running tier1.
> 
> Thanks,
> Johan

This pull request has now been integrated.

Changeset: 97e99f01
Author:    Johan Sjölen <jsjolen at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/97e99f01d4f317fce1a204c01874a68f5e25a051
Stats:     24 lines in 1 file changed: 4 ins; 0 del; 20 mod

8311087: PhiNode::wait_for_region_igvn should break early

Reviewed-by: thartmann, chagedorn

-------------

PR: https://git.openjdk.org/jdk/pull/14707


More information about the hotspot-compiler-dev mailing list