On Tue, 2 Dec 2025 13:24:54 GMT, Coleen Phillimore <[email protected]> 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
