RFR: 8321278: C2: Partial peeling fails with assert "last_peel <- first_not_peeled" [v3]

[Roland Westrelin]

On Fri, 22 Mar 2024 16:01:46 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

>> Roland Westrelin has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   review
>
> test/hotspot/jtreg/compiler/loopopts/TestPartialPeelingAtSingleInputRegion.java line 51:
> 
>> 49: 
>> 50:     public static void main(String[] args) {
>> 51:         for (int i = 0; i < 50_000; ++i) {
> 
> Just a minor thing: Do you really need 50000 iterations or would fewer be sufficient to trigger the bug?

Good catch. It reproduces with 10000. I pushed a comment that makes that change.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18353#discussion_r1535880857