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

Paul Hübner phubner at openjdk.org
Tue Dec 16 10:19:23 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 incrementally with one additional commit since the last revision:

  Reviewer comments.

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

Changes:
  - all: https://git.openjdk.org/valhalla/pull/1759/files
  - new: https://git.openjdk.org/valhalla/pull/1759/files/58fb031a..e6c091fb

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=valhalla&pr=1759&range=03
 - incr: https://webrevs.openjdk.org/?repo=valhalla&pr=1759&range=02-03

  Stats: 4 lines in 4 files changed: 0 ins; 1 del; 3 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