RFR: 8369258: C2: enable ReassociateInvariants for all loop types [v2]
    Quan Anh Mai 
    qamai at openjdk.org
       
    Wed Oct 15 09:42:33 UTC 2025
    
    
  
On Wed, 15 Oct 2025 09:38:30 GMT, Quan Anh Mai <qamai 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 four additional commits since the last revision:
>> 
>>  - review
>>  - Merge branch 'master' into JDK-8369258
>>  - test fixes
>>  - test and fix
>
> test/hotspot/jtreg/compiler/loopopts/TestReassociateInvariants.java line 63:
> 
>> 61:     @IR(failOn = { IRNode.COUNTED_LOOP, IRNode.LONG_COUNTED_LOOP })
>> 62:     @IR(counts = { IRNode.LOOP, "1" })
>> 63:     @Arguments(values = { Argument.NUMBER_42, Argument.NUMBER_42 })
> 
> What are we verifying here, should this fail on some kind of range check?
Also, why are these not recognized as `CountedLoop`s, it seems we need to fix them, too?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27666#discussion_r2431870029
    
    
More information about the hotspot-compiler-dev
mailing list