RFR: 8356708: C2: loop strip mining expansion doesn't take sunk stores into account [v3]

[Roland Westrelin]

On Wed, 11 Jun 2025 14:13:33 GMT, Roberto Castañeda Lozano <rcastanedalo 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 six additional commits since the last revision:
>> 
>>  - reviews
>>  - Merge branch 'master' into JDK-8356708
>>  - Update src/hotspot/share/opto/loopnode.cpp
>>    
>>    Co-authored-by: Emanuel Peter <emanuel.peter at oracle.com>
>>  - Update src/hotspot/share/opto/loopnode.cpp
>>    
>>    Co-authored-by: Roberto Castañeda Lozano <robcasloz at users.noreply.github.com>
>>  - Update src/hotspot/share/opto/loopnode.cpp
>>    
>>    Co-authored-by: Roberto Castañeda Lozano <robcasloz at users.noreply.github.com>
>>  - test & fix
>
> test/hotspot/jtreg/compiler/loopstripmining/TestStoresSunkInOuterStripMinedLoop.java line 1:
> 
>> 1: /*
> 
> It would be good, for completeness, to add a "Couple stores sunk in outer loop, store in inner loop" test.

Done in the new commit.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25717#discussion_r2143082613