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

Reply via email to