RFR: 8316533: C2 compilation fails with assert(verify(phase)) failed: missing Value() optimization [v4]

[Emanuel Peter]

On Fri, 3 Nov 2023 10:43:21 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

>> Emanuel Peter has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 14 additional commits since the last revision:
>> 
>>  - Merge branch 'master' into JDK-8316533
>>  - Merge branch 'master' into JDK-8316533
>>  - Merge branch 'JDK-8316533' of https://github.com/eme64/jdk into JDK-8316533
>>  - Apply suggestions from code review
>>    
>>    From Roberto
>>    
>>    Co-authored-by: Roberto Castañeda Lozano <robcasloz at users.noreply.github.com>
>>  - fix type on class name, spotted by Roberto
>>  - Merge branch 'master' into JDK-8316533
>>  - remove CmpP and more comments
>>  - return EQ instead of TOP
>>  - CmpP null-check solution
>>  - add ignore flag to test
>>  - ... and 4 more: https://git.openjdk.org/jdk/compare/2ca56414...808e2c38
>
> Looks good to me. @rwestrel should take a look as well.

Thanks @TobiHartmann  @robcasloz @rwestrel for the reviews!

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

PR Comment: https://git.openjdk.org/jdk/pull/16361#issuecomment-1794532568