On Tue, 2 Jun 2026 06:20:23 GMT, Tobias Hartmann <[email protected]> wrote:

>> I propose transforming:
>> 
>> (InlineType .... (InlineType oop
>> 
>> 
>> where an `InlineType` references another `InlineType` node (maybe itself) 
>> through its `oop` input and with `Phi` or cast nodes in the subgraph in 
>> between into:
>> 
>> 
>> (InlineType .... oop
>> 
>> which requires cloning every cast or `Phi` nodes in the subgraph between the 
>> 2 `InlineType` nodes. That prevent sinfinite invocation of 
>> `PhiNode::push_inline_types_through` and cleans up the graph.
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> src/hotspot/share/opto/cfgnode.hpp line 183:
> 
>> 181:   bool must_wait_for_region_in_irreducible_loop(PhaseGVN* phase) const;
>> 182: 
>> 183:   bool can_push_inline_types_down(PhaseGVN* phase, bool can_reshape, 
>> ciInlineKlass*& inline_klass);
> 
> This is dead, right?

Right.

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

PR Review Comment: 
https://git.openjdk.org/valhalla/pull/2316#discussion_r3341186662

Reply via email to