[lworld] Integrated: 8371357: [lworld] Remove EnableValhalla

Paul Hübner phubner at openjdk.org
Wed Dec 17 10:52:58 UTC 2025


On Tue, 25 Nov 2025 13:37:46 GMT, Paul Hübner <phubner at openjdk.org> wrote:

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

This pull request has now been integrated.

Changeset: 59a270d7
Author:    Paul Hübner <phubner at openjdk.org>
URL:       https://git.openjdk.org/valhalla/commit/59a270d73a2cc067bdc8c3380eafc615e1284481
Stats:     159 lines in 56 files changed: 59 ins; 13 del; 87 mod

8371357: [lworld] Remove EnableValhalla

Reviewed-by: coleenp, stefank, dcubed

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

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


More information about the valhalla-dev mailing list