[lworld] RFR: 8371357: [lworld] Remove EnableValhalla
Paul Hübner
phubner at openjdk.org
Fri Dec 12 16:21:36 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.
-------------
Commit messages:
- Reviewer comments.
- Copyright.
- Revert "Don't synchronize on Double."
- Accidentally changed too many enable_preview.
- Make heap dump test require debug VM.
- Change to arguments.
- Use is_valhalla_enabled.
- Remove unused file.
- Reset problem lists.
- Explode JDK groups and problemlist tests.
- ... and 9 more: https://git.openjdk.org/valhalla/compare/c6706524...e8e771a9
Changes: https://git.openjdk.org/valhalla/pull/1759/files
Webrev: https://webrevs.openjdk.org/?repo=valhalla&pr=1759&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8371357
Stats: 157 lines in 55 files changed: 59 ins; 13 del; 85 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