On Mon, 18 May 2026 09:22:36 GMT, Tobias Hartmann <[email protected]> wrote:

> This patch re-enables the `EliminateAutoBox` functionality when 
> `--enable-preview` is set, i.e. when the primitive boxes are value classes. 
> `EliminateAutoBox` has multiple effects, including an impact on inlining 
> decisions but its main goal is to avoid boxing via `Integer.valueOf(int)` etc 
> by replacing that call. I added support for scalarized returns from that 
> method and also added support for removing unboxing via `Integer.intValue` 
> etc.
> 
> With this fix, I'm seeing various improvements:
> 
> <img width="570" height="513" alt="Screenshot from 2026-06-02 11-08-53" 
> src="https://github.com/user-attachments/assets/7c62b35b-a8ea-44ce-aa5c-d6a66f113973";
>  />
> 
> (The regressions are not significant and due to high variance of the 
> benchmark)
> 
> There's more we could do here but I already spent way too much time on this, 
> so I'll defer follow-up work to 
> [JDK-8385809](https://bugs.openjdk.org/browse/JDK-8385809).
> 
> Thanks,
> Tobias
> 
> ---------
> - [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/macro.cpp line 707:

> 705: 
> 706:   // Create a new InlineTypeNode and retrieve the field values from 
> memory
> 707:   InlineTypeNode* vt = InlineTypeNode::make_uninitialized(_igvn, vk, 
> null_free);

Just spotted this when browsing the code.

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

PR Review Comment: 
https://git.openjdk.org/valhalla/pull/2446#discussion_r3340129486

Reply via email to