Integrated: 8263075: C2: simplify anti-dependence check in PhaseCFG::implicit_null_check()

[Brian J. Stafford]

On Thu, 12 May 2022 16:48:40 GMT, Brian J. Stafford <duke at openjdk.java.net> wrote:

> The reporter for this issue (https://bugs.openjdk.java.net/browse/JDK-8263075) indicated that there's an assumption that we can rely on that the while loop in question will run exactly one time. Based on this, I've done the following:
> 
> - Asserted the condition that makes sure the code runs at least once
> - Asserted the condition that makes sure the code runs only once
> - Removed the `while` loop
> - Changed a couple of `break` statements into `continue` statements. They no longer need to break out of the `while` loop, now that it's gone. However, they were early exits from the `while` loop that ended up resulting in `continue` statements for the larger enclosing loop. Thus we can just call `continue` directly.
> - Removed the local variable `b`, as we no longer need to traverse the node hierarchy. We can use `mb` directly.
> 
> Passes jdk, langtools, and hotspot Tier 1 tests on Linux (x64 and ARM64) and macOS (x64 and ARM64). Most Tier 1 tests pass on Windows (x64 and ARM64), but there are a handful of failures unrelated to this change.

This pull request has now been integrated.

Changeset: c6743489
Author:    Brian J. Stafford <brianjstafford at gmail.com>
Committer: Vladimir Kozlov <kvn at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/c6743489d2fb65f3fe05b403ae66ac30e6aa4846
Stats:     21 lines in 1 file changed: 2 ins; 0 del; 19 mod

8263075: C2: simplify anti-dependence check in PhaseCFG::implicit_null_check()

Reviewed-by: kvn, thartmann, rcastanedalo

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

PR: https://git.openjdk.java.net/jdk/pull/8684