On Thu, 3 Jul 2025 00:09:14 GMT, Ioi Lam <ik...@openjdk.org> wrote:

>> Also quick question: Should I use `_transitive_interfaces` or can I use 
>> `_local_interfaces`?
>
> local_interfaces is fine, because the interfaces implemented by the super 
> classes would have been checked when the super classes were loaded.
> 
> BTW, the error message should include the name of both this class and the 
> supertype:
> 
> 
> classfile_parse_error("AOTSafeClassInitlaizer annotation is required for 
> supertype %s of %s", ...
> 
> 
> And the checks can be refactored in a separate function to avoid repetition.
> 
> 
> if (_super_klass != nullptr) {
>   check_aot_safe_initializer(ik, _super_klass, CHECK);
> }
> int len = _local_interfaces->length();
> for (int i = 0; i < len; i++) {
>   check_aot_safe_initializer(ik, _local_interfaces->at(i), CHECK);
> }

I don't want to introduce a new declaration in the header for this function - 
to shrink the size, I think I can use `guarantee_property` instead.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25922#discussion_r2182930135

Reply via email to