[lworld] RFR: 8371357: [lworld] Remove EnableValhalla
David Holmes
dholmes at openjdk.org
Mon Dec 15 06:45:25 UTC 2025
On Fri, 12 Dec 2025 16:07:09 GMT, Paul Hübner <phubner at openjdk.org> wrote:
> is_valhalla_enabled
As Ioi suggested in the mainline argument.hpp PR shouldn't this be something more specific to what is actually being enabled in this preview release e.g. valueTypesConfig.hpp which defines `value_types_enabled` or some such? IIUC for this release we are talking about JEP-401 features that are opted into when `--enable-preview` is added. But for the next release I assume there could be additional JEPs with 401 still also in preview - so `is_valhalla_enabled` is somewhat vague and doesn't identify what aspect of Valhalla a piece of code relates to.
-------------
PR Comment: https://git.openjdk.org/valhalla/pull/1759#issuecomment-3653729866
More information about the valhalla-dev
mailing list