[lworld] RFR: 8294462: [lworld] set system property valhalla.enableValhalla when -XX:+EnableValhalla [v2]
Mandy Chung
mchung at openjdk.org
Tue Sep 27 17:56:45 UTC 2022
On Tue, 27 Sep 2022 15:04:22 GMT, Roger Riggs <rriggs at openjdk.org> wrote:
>> Updated to set the system property after all of the option setting locations have been processed.
>> Verified via -XshowSettings.
>
> Roger Riggs has updated the pull request incrementally with one additional commit since the last revision:
>
> Remove extra blank line
We need an internal API to test if `-XX:+EnableValhalla` and `-XX:+EnablePrimitiveClass` flags are set. Instead of using a system property, it may be time to add a method in `jdk.internal.value.PrimitiveClass` or other class to ask the VM if the flag is enabled.
A side note: `-XX:+EnableValhalla` will ultimately become `--enable-preview` and the internal API `jdk.internal.misc.PreviewFeatures::isEnabled` already exists.
-------------
PR: https://git.openjdk.org/valhalla/pull/769
More information about the valhalla-dev
mailing list