> 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).

Tobias Hartmann has updated the pull request incrementally with one additional 
commit since the last revision:

  Move logging related code

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

Changes:
  - all: https://git.openjdk.org/valhalla/pull/2446/files
  - new: https://git.openjdk.org/valhalla/pull/2446/files/7e7caea2..f1cfa947

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=valhalla&pr=2446&range=01
 - incr: https://webrevs.openjdk.org/?repo=valhalla&pr=2446&range=00-01

  Stats: 21 lines in 1 file changed: 10 ins; 11 del; 0 mod
  Patch: https://git.openjdk.org/valhalla/pull/2446.diff
  Fetch: git fetch https://git.openjdk.org/valhalla.git pull/2446/head:pull/2446

PR: https://git.openjdk.org/valhalla/pull/2446

Reply via email to