[lworld] RFR: 8304168: [lworld] CDS tests fail with --enable-preview patched value classes

Ioi Lam iklam at openjdk.org
Tue Mar 26 18:54:39 UTC 2024


On Tue, 26 Mar 2024 18:42:41 GMT, Roger Riggs <rriggs at openjdk.org> wrote:

> Can we drop `EnableValhalla`; I believe we're trying to remove that and use only --enable-preview.

There's existing code in arguments.cpp that checks `if (enable_preview() && EnableValhalla)`. I think we can clean that up in a separate PR.

https://github.com/openjdk/valhalla/blob/a226fb4bff53a715acc83ef56187d5e174c10ed0/src/hotspot/share/runtime/arguments.cpp#L2195-L2201

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

PR Comment: https://git.openjdk.org/valhalla/pull/1063#issuecomment-2021234249



More information about the valhalla-dev mailing list