RFR: 8338100: C2: assert(!n_loop->is_member(get_loop(lca))) failed: control must not be back in the loop [v3]

Roland Westrelin roland at openjdk.org
Thu Sep 5 15:53:59 UTC 2024


On Thu, 5 Sep 2024 13:34:42 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 nine additional commits since the last revision:
>> 
>>  - remove useless PhaseIdealLoop::only_has_infinite_loops()
>>  - Merge branch 'master' into JDK-8338100
>>  - Update src/hotspot/share/opto/loopnode.hpp
>>    
>>    Co-authored-by: Tobias Hartmann <tobias.hartmann at oracle.com>
>>  - Update src/hotspot/share/opto/loopnode.cpp
>>    
>>    Co-authored-by: Christian Hagedorn <christian.hagedorn at oracle.com>
>>  - Update src/hotspot/share/opto/loopnode.cpp
>>    
>>    Co-authored-by: Christian Hagedorn <christian.hagedorn at oracle.com>
>>  - comment
>>  - test fix
>>  - remove verification code
>>  - test & fix
>
> Looks good.

Thanks for the re-reviews @TobiHartmann @chhagedorn

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

PR Comment: https://git.openjdk.org/jdk/pull/20797#issuecomment-2332091249


More information about the hotspot-compiler-dev mailing list