RFR: 8345299: C2: some nodes can still have incorrect control after do_range_check() [v2]

[Roland Westrelin]

On Tue, 3 Dec 2024 17:14:28 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>> Roland Westrelin has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   review
>
> src/hotspot/share/opto/loopTransform.cpp line 2825:
> 
>> 2823:   // new main_limit can push Bool/Cmp nodes down (when one of the eliminated condition has parameters that are not loop
>> 2824:   // invariant in the pre loop).
>> 2825:   set_ctrl(opqzm, new_limit_ctrl);
> 
> Update this comment too.

Both done in new commit. Do the comments look ok to you?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22485#discussion_r1869756543