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

Chen Liang liach at openjdk.org
Thu Jul 24 17:10:11 UTC 2025


On Thu, 24 Jul 2025 12:32:52 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> 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 one additional commit since the last revision:
> 
>   Fix the test some more.

test/jdk/valhalla/valuetypes/IsIdentityClassTest.java line 30:

> 28:  * @enablePreview false
> 29:  * @modules java.base/jdk.internal.misc
> 30:  *          java.base/jdk.internal.value

TEST.properties is updated; we might remove these directives.

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

PR Review Comment: https://git.openjdk.org/valhalla/pull/1514#discussion_r2229088882


More information about the valhalla-dev mailing list