[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