RFR: 8372098: Move AccessFlags to InstanceKlass [v4]

Vladimir Ivanov vlivanov at openjdk.org
Mon Nov 24 22:59:36 UTC 2025


On Fri, 21 Nov 2025 14:53:03 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:
> 
>   Reformatting compile.cpp

src/hotspot/share/opto/library_call.cpp line 4100:

> 4098:     // Other types can report the actual _super.
> 4099:     // (To verify this code sequence, check the asserts in JVM_IsInterface.)
> 4100:     if (generate_interface_guard(kls, region) != nullptr)

BTW why did you decide to change the order of the checks?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28371#discussion_r2557971918


More information about the graal-dev mailing list