RFR: 8355488: Add stress mode for C2 loop peeling [v3]

Tobias Hartmann thartmann at openjdk.org
Fri May 16 08:34:56 UTC 2025


On Thu, 15 May 2025 13:48:53 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

>> src/hotspot/share/opto/loopnode.hpp line 137:
>> 
>>> 135: 
>>> 136: #ifndef PRODUCT
>>> 137:   uint _peeling_opportunities_count = 0;
>> 
>> I think this should rather be named `_stress_peeling_attempts` or something.
>
> I thought about proposing "attempts" as well. But it sounds like we are actually trying loop peeling and then somehow fail which is not the case here - we just flipped a coin and then decided not to do peeling. Anyway, I don't have a strong opinion here :-)

Right, it's not a perfect name either but let's not bikeshed too much - I'm fine either way.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25140#discussion_r2092595184


More information about the hotspot-compiler-dev mailing list