[lworld] RFR: 8281463: [lworld] VALUE / PRIMITIVE modifiers should be supported by reflection [v7]

Roger Riggs rriggs at openjdk.org
Fri Jun 17 17:26:37 UTC 2022


> Add support to java.lang.reflect.AccessFlag ACC_IDENTITY, ACC_VALUE, and ACC_PRIMITIVE.
> Added corresponding flags to j.l.reflect.Modifier so the "source" of the AccessFlag is the Modifer mask.
> Remove PERMITS_VALUE; moving the definition used by HotSpot to HotSpot code.
> 
> Add comments to java.l.reflect.Modifier suggesting use of AccessFlag instead and limitations.

Roger Riggs has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains ten additional commits since the last revision:

 - Merge branch 'lworld' into 8281463-value-modifiers
 - Proxy class definition needs ACC_IDENTITY
   Restore ACC_SUPER to AccessFlag and adjust test
 - Replace static int constants with AccessFlag method calls.
 - Revert changes and simplify
   Update GenericStringTest to work with value and primitive classes.
 - Address review comments
 - Merge branch 'lworld' into 8281463-value-modifiers
 - Add Class.is(AccessFlag) to test for a single flag.
 - Restore AccessFlag SUPER
 - 8281463: [lworld] VALUE / PRIMITIVE modifiers should be supported by reflection
   Use java.lang.reflect.AccessFlag for equivalent functions for value objects.
   Corrected ModuleDescriptor.STATIC flags to be STATIC_PHASE
   Added flags to j.l.reflect.Modifier.

-------------

Changes:
  - all: https://git.openjdk.org/valhalla/pull/698/files
  - new: https://git.openjdk.org/valhalla/pull/698/files/349a9399..558d8734

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=valhalla&pr=698&range=06
 - incr: https://webrevs.openjdk.org/?repo=valhalla&pr=698&range=05-06

  Stats: 708 lines in 49 files changed: 343 ins; 99 del; 266 mod
  Patch: https://git.openjdk.org/valhalla/pull/698.diff
  Fetch: git fetch https://git.openjdk.org/valhalla pull/698/head:pull/698

PR: https://git.openjdk.org/valhalla/pull/698



More information about the valhalla-dev mailing list