[lworld] RFR: 8363846: [lworld] Make Class.isIdentityClass() non-native [v2]
Coleen Phillimore
coleenp at openjdk.org
Wed Jul 23 12:15:06 UTC 2025
On Tue, 22 Jul 2025 23:44:21 GMT, Chen Liang <liach at openjdk.org> wrote:
> is_identity_class seems widely used in many areas, such as C2 - do we need to verify those usages?
Good question. I did check most/attempted all of these places and added the assert in klass.hpp but since AccessFlags are in klass, it is possible to copy them from an ArrayKlass to a ciKlass and fail the is_identity check for an array. The only way to make this completely safe is to move the access flags, which I started in another repo but it's got pretty hairy since Klass and InstanceKlass are used interchangeably in some of the compiler code that I don't know very well, when I think they really mean InstanceKlass.
-------------
PR Comment: https://git.openjdk.org/valhalla/pull/1514#issuecomment-3107679225
More information about the valhalla-dev
mailing list