RFR: 8342692: C2: long counted loop/long range checks: don't create loop-nest for short running loops [v20]
Christian Hagedorn
chagedorn at openjdk.org
Wed May 21 08:47:01 UTC 2025
On Thu, 15 May 2025 15:26:34 GMT, Roland Westrelin <roland at openjdk.org> wrote:
>> src/hotspot/share/opto/loopnode.cpp line 1269:
>>
>>> 1267: }
>>> 1268: #endif
>>> 1269: entry_control = head->skip_strip_mined()->in(LoopNode::EntryControl);
>>
>> It looks like this line rather belongs to the `Predicate` on L1275? Might have been moved here by accident.
>
> I don't think that's the case. Predicates were added so `entry_control` needs to be refreshed.
You're right, I thought we need the skipping of strip mined loop below as well. But we don't have a strip mined loop for the long case - so all good.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21630#discussion_r2099725506
More information about the hotspot-compiler-dev
mailing list