RFR: 8372098: Move AccessFlags to InstanceKlass [v7]

Serguei Spitsyn sspitsyn at openjdk.org
Tue Dec 2 19:23:36 UTC 2025


On Tue, 2 Dec 2025 13:24:54 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> ArrayKlass doesn't set AccessFlags so don't look for them there.  See CR for details.
>> Fixed SA and jvmci.  @iwanowww Can you check that I changed C2 correctly (we talked about this in August).
>> Tested with tier1-4.  5-7 in progress.
>
> Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Fixed indentation.

Marked as reviewed by sspitsyn (Reviewer).

src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/oops/InstanceKlass.java line 101:

> 99:     }
> 100:     headerSize           = type.getSize();
> 101:     accessFlags  = new CIntField(type.getCIntegerField("_access_flags"), 0);

Nit: Not a big deal but this line also does not have proper indent. You can ignore this comment though. :)

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

PR Review: https://git.openjdk.org/jdk/pull/28371#pullrequestreview-3531778810
PR Review Comment: https://git.openjdk.org/jdk/pull/28371#discussion_r2582521355


More information about the serviceability-dev mailing list