RFR: 8315920: C2: "control input must dominate current control" assert failure [v3]

[Roland Westrelin]

On Fri, 15 Sep 2023 13:47:36 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

>> Roland Westrelin 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 seven additional commits since the last revision:
>> 
>>  - comments
>>  - Merge branch 'master' into JDK-8315920
>>  - Update src/hotspot/share/opto/loopopts.cpp
>>    
>>    Co-authored-by: Christian Hagedorn <christian.hagedorn at oracle.com>
>>  - Update src/hotspot/share/opto/loopopts.cpp
>>    
>>    Co-authored-by: Tobias Hartmann <tobias.hartmann at oracle.com>
>>  - Update src/hotspot/share/opto/loopopts.cpp
>>    
>>    Co-authored-by: Tobias Hartmann <tobias.hartmann at oracle.com>
>>  - Update src/hotspot/share/opto/loopopts.cpp
>>    
>>    Co-authored-by: Tobias Hartmann <tobias.hartmann at oracle.com>
>>  - fix & test
>
> This looks good to me but needs some more comments.

@TobiHartmann @chhagedorn thanks for reviewing this. I pushed a new commit with comments. Let me know if that looks ok to you.

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

PR Comment: https://git.openjdk.org/jdk/pull/15720#issuecomment-1733732840