On Wed, 24 Jun 2026 11:20:25 GMT, Roland Westrelin <[email protected]> wrote:
>> src/hotspot/share/opto/callGenerator.cpp line 187:
>>
>>> 185: kit.set_edges_for_java_call(call, false, _separate_io_proj);
>>> 186: Node* ret = kit.set_results_for_java_call(call, _separate_io_proj);
>>> 187: if (is_late_inline() && !call->is_boxing_method() && ret->is_Proj()
>>> && ret->in(0) == call) {
>>
>> Why do you need `!call->is_boxing_method()` here?
>
> That was in the mainline fix:
> https://github.com/openjdk/jdk/pull/30088
>
> The reason if I remember correctly is that `Node::is_dead_loop_safe()` has
> some custom logic for `is_boxing_method()`
Oh, I overlooked it's a backport. Sorry for the noise.
-------------
PR Review Comment:
https://git.openjdk.org/valhalla/pull/2575#discussion_r3470559827
