RFR: 8369898: C2 SuperWord: assert(has_ctrl(i)) failed: should be control, not loop [v2]

[Emanuel Peter]

On Mon, 20 Oct 2025 13:03:18 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

>> Emanuel Peter has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Apply suggestions from code review
>>   
>>   Co-authored-by: Christian Hagedorn <christian.hagedorn at oracle.com>
>
> The fix looks reasonable to me and I agree that we should improve the documentation for `lazy_replace()` and also think about it's naming - by just looking at the name, one could first guess that the actual node replacement is delayed which is not the case! 
> 
> Maybe @rwestrel also wants to have a look who worked with `lazy_replace()` in the mentioned PR https://github.com/openjdk/jdk/pull/15720.

@chhagedorn Thanks for the suggestions, they are all applied :)

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

PR Comment: https://git.openjdk.org/jdk/pull/27889#issuecomment-3422096168