RFR: 8334544: C2: wrong control assigned in PhaseIdealLoop::clone_assertion_predicate_for_unswitched_loops()

Wed Jun 19 09:26:09 UTC 2024

On Wed, 19 Jun 2024 08:26:25 GMT, Roland Westrelin <roland at openjdk.org> wrote:

> There's a minor (and harmless AFAICT) bug in
> `PhaseIdealLoop::clone_assertion_predicate_for_unswitched_loops()`. Control
> for the new predicate expression is set to `parse_predicate_proj` but
> the `If` for the new predicate is created to be above
> `parse_predicate_proj`.

Looks good to me as well.

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

Marked as reviewed by thartmann (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/19787#pullrequestreview-2127645429