[lworld] RFR: 8371357: [lworld] Remove EnableValhalla [v3]

Paul Hübner phubner at openjdk.org
Mon Dec 15 19:46:03 UTC 2025


> Hi all,
> 
> This removes the `EnableValhalla` in favour of the `--enable-preview` flag. Concretely:
> * I've replaced most of the `EnableValhalla` checks with `Arguments::is_valhalla_enabled()`. 
> * Some checks were redundant and could be removed entirely.
> * I've made the `EnableValhalla` flag obsolete.
> * Some tests had to be updated.
> 
> This greatly changes the semantics of tests. I've refined some test groups to make it easier.
> 
> Testing: tiers 1-4.

Paul Hübner has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 23 commits:

 - Use the new function for the Parallel changes.
 - Merge branch 'lworld' into JDK-8371357
 - Remove wrong comment.
 - Fix imports.
 - Reviewer comments.
 - Copyright.
 - Revert "Don't synchronize on Double."
   
   This reverts commit cbf849c2ce6059af2b43b3dd8cb42445eab4df2b.
 - Accidentally changed too many enable_preview.
 - Make heap dump test require debug VM.
 - Change to arguments.
 - ... and 13 more: https://git.openjdk.org/valhalla/compare/8c0c190b...58fb031a

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

Changes: https://git.openjdk.org/valhalla/pull/1759/files
  Webrev: https://webrevs.openjdk.org/?repo=valhalla&pr=1759&range=02
  Stats: 157 lines in 56 files changed: 59 ins; 12 del; 86 mod
  Patch: https://git.openjdk.org/valhalla/pull/1759.diff
  Fetch: git fetch https://git.openjdk.org/valhalla.git pull/1759/head:pull/1759

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


More information about the valhalla-dev mailing list