[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