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

Thu Jun 20 07:16:15 UTC 2024

On Wed, 19 Jun 2024 09:10:34 GMT, Christian Hagedorn <chagedorn 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`.
>
> Good catch! Yes, should be harmless. Looks good.

@chhagedorn @TobiHartmann thanks for the reviews

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

PR Comment: https://git.openjdk.org/jdk/pull/19787#issuecomment-2179978112