[lworld] RFR: 8303137: [lworld] Fix ArchivedEnumTest fail caused by changed Enum static field [v2]
Mandy Chung
mchung at openjdk.org
Fri Feb 24 22:53:30 UTC 2023
On Fri, 24 Feb 2023 20:10:18 GMT, Roger Riggs <rriggs at openjdk.org> wrote:
>> Removed private static field in j.l.reflect.AccesFlag that CDS could not archive.
>> Replaced with inline expression.
>
> Roger Riggs has updated the pull request incrementally with one additional commit since the last revision:
>
> Remove static field initialization based on ValhallaFeatures enabled.
src/java.base/share/classes/java/lang/reflect/AccessFlag.java line 669:
> 667: Set.of(CLASS, INNER_CLASS);
> 668: private static final Set<Location> SET_CLASS_IDENTITY_VALHALLA =
> 669: ValhallaFeatures.isEnabled() ? SET_CLASS_INNER_CLASS : EMPTY_SET;
Should this static final field be removed?
-------------
PR: https://git.openjdk.org/valhalla/pull/827
More information about the valhalla-dev
mailing list