RFR: JDK-8287284: C2: loop optimization performs split_thru_phi infinitely many times [v9]

Mon Nov 13 10:16:05 UTC 2023

On Fri, 10 Nov 2023 10:56:30 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> Tobias Holenstein has updated the pull request incrementally with three additional commits since the last revision:
>> 
>>  - Update src/hotspot/share/opto/loopopts.cpp
>>    
>>    Co-authored-by: Christian Hagedorn <christian.hagedorn at oracle.com>
>>  - Merge branch '8287284' of github.com:tobiasholenstein/jdk into 8287284
>>  - TestSplitThruPhiInfinitely
>
> test/hotspot/jtreg/compiler/loopopts/TestSplitThruPhiInfinitely.java line 33:
> 
>> 31:  * @run main/othervm -Xbatch -XX:-PartialPeelLoop -XX:CompileCommand=compileonly,compiler.c2.loopopts.TestSplitThruPhiInfinitely::run* compiler.c2.loopopts.TestSplitThruPhiInfinitely
>> 32: 
>> 33: 
> 
> You should have a `*` on every line.

resolved now

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15536#discussion_r1390901995