[lworld] RFR: 8363846: [lworld] Make Class.isIdentityClass() non-native [v5]

Coleen Phillimore coleenp at openjdk.org
Wed Jul 23 19:02:22 UTC 2025


> This moves isIdentityClass() implementation to Class.java, and checks within the JVM that is_identity_class() doesn't check the access flags for an ArrayKlass, since AccessFlags aren't initialized in ArrayKlasses.  The AccessFlags should be moved from Klass.hpp to InstanceKlass.cpp in mainline but that's a more complicated change and has several pieces.
> Added a test.
> Tested with tier1 locally.

Coleen Phillimore has updated the pull request incrementally with three additional commits since the last revision:

 - Update test/jdk/valhalla/valuetypes/IsIdentityClassTest.java
   
   Co-authored-by: Roger Riggs <Roger.Riggs at Oracle.com>
 - Update test/jdk/valhalla/valuetypes/IsIdentityClassTest.java
   
   Co-authored-by: Roger Riggs <Roger.Riggs at Oracle.com>
 - Add @enablePreview false and import.

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

Changes:
  - all: https://git.openjdk.org/valhalla/pull/1514/files
  - new: https://git.openjdk.org/valhalla/pull/1514/files/18f90e4f..3760602f

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=valhalla&pr=1514&range=04
 - incr: https://webrevs.openjdk.org/?repo=valhalla&pr=1514&range=03-04

  Stats: 8 lines in 1 file changed: 4 ins; 1 del; 3 mod
  Patch: https://git.openjdk.org/valhalla/pull/1514.diff
  Fetch: git fetch https://git.openjdk.org/valhalla.git pull/1514/head:pull/1514

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


More information about the valhalla-dev mailing list