RFR: 8360163: Create annotations to mark dumping method handles and runtime setup required classes [v8]
Chen Liang
liach at openjdk.org
Thu Jul 3 14:26:42 UTC 2025
On Thu, 3 Jul 2025 00:09:14 GMT, Ioi Lam <iklam at 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
More information about the hotspot-dev
mailing list