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