On Wed, 2 Jul 2025 21:18:14 GMT, Ioi Lam <ik...@openjdk.org> wrote:
>> Chen Liang has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Documentation
>
> src/hotspot/share/classfile/classFileParser.cpp line 5171:
>
>> 5169: InstanceKlass* intf = _transitive_interfaces->at(i);
>> 5170: if (intf->class_initializer() != nullptr) {
>> 5171: if (!intf->has_aot_safe_initializer()) {
>
> I think this is better, so you don't need to annotate a supertypes that have
> no `<clinit>`
>
>
> if (intf->class_initializer() != nullptr &&
> !intf->has_aot_safe_initializer()) {
Also quick question: Should I use `_transitive_interfaces` or can I use
`_local_interfaces`?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25922#discussion_r2181136140
