RFR: 8342692: C2: long counted loop/long range checks: don't create loop-nest for short running loops [v15]

[Roland Westrelin]

On Wed, 23 Apr 2025 09:18:27 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> Roland Westrelin has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 41 commits:
>> 
>>  - merge fix
>>  - Merge branch 'master' into JDK-8342692
>>  - merge fix
>>  - Merge branch 'master' into JDK-8342692
>>  - merge
>>  - Merge branch 'master' into JDK-8342692
>>  - Merge branch 'master' into JDK-8342692
>>  - whitespace
>>  - Merge branch 'master' into JDK-8342692
>>  - TestMemorySegment test fix
>>  - ... and 31 more: https://git.openjdk.org/jdk/compare/dc5c4148...065abb29
>
> test/hotspot/jtreg/compiler/rangechecks/TestLongRangeCheck.java line 37:
> 
>> 35:  * @run main/othervm -ea -Xbootclasspath/a:. -XX:+UnlockDiagnosticVMOptions -XX:+WhiteBoxAPI -XX:-BackgroundCompilation -XX:-UseOnStackReplacement
>> 36:  *                   -XX:+UnlockExperimentalVMOptions -XX:PerMethodSpecTrapLimit=5000 -XX:PerMethodTrapLimit=100 -XX:+IgnoreUnrecognizedVMOptions
>> 37:  *                   -XX:-StressShortRunningLongLoop
> 
> Why was this necessary? A comment could be nice.

That one doesn't appear to be needed anymore. I removed it.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21630#discussion_r2077609428