On Tue, 30 Jun 2026 08:22:27 GMT, Christian Hagedorn <[email protected]>
wrote:
>> Tobias Hartmann has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Handle recursion
>
> src/hotspot/share/opto/phaseX.cpp line 2491:
>
>> 2489: void PhaseIterGVN::replace_in_uses(Node* n, Node* m) {
>> 2490: assert(n != nullptr, "sanity");
>> 2491: add_users_to_worklist(n);
>
> This is probably also beneficial for existing uses of `replace_in_uses()`.
Yes, I think so too.
-------------
PR Review Comment:
https://git.openjdk.org/valhalla/pull/2593#discussion_r3497290552
