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

[Emanuel Peter]

On Fri, 9 May 2025 12:38:03 GMT, Roland Westrelin <roland at openjdk.org> wrote:

>>> Interesting work and results with the benchmark! I have a few comments. Will have another look again later.
>> 
>> Thanks for reviewing this @chhagedorn 
>> I think new commit addresses all your comments.
>
>> @rwestrel Sorry it took so long for me to look at this.
>> 
>> Did you do some benchmarking to prove that `ShortLoopIter = 1000` is reasonable?
> 
> As mentioned in one of my replies to your questions, I removed that command line argument.
> 
>> The Benchmark you published earlier does not seem to do that, right? [#21630 (comment)](https://github.com/openjdk/jdk/pull/21630#issuecomment-2587016221)
>> 
>> I would also like to see that benchmark integrated. If you are using a benchmark to demonstrate the performance, it should be integrate so others can easily verify on their platform :)
> 
> Right. But it's Maurizio's benchmark. I think it would make sense to integrate it separately. What do you think @mcimadamore ?

@rwestrel The PR description is now out of sync with what you are doing. For example, it still mentions `ShortLoopIter`. It would also be nice if the benchmark results were in the PR description. At least links to the relevant mesasges below, because otherwise the most relevant information is burried somewhere in the "hidden items".

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

PR Comment: https://git.openjdk.org/jdk/pull/21630#issuecomment-2890256755