RFR: 8293337: Store method handle intrinsics in AOT cache [v2]

Ioi Lam iklam at openjdk.org
Tue Sep 17 00:06:19 UTC 2024


On Fri, 13 Sep 2024 21:45:45 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>> Ioi Lam has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains nine additional commits since the last revision:
>> 
>>  - @vnkozlov comment - added NOT_CDS_RETURN
>>  - Merge branch 'jep-483-step-04-8293187-support-sun-invoke-util-wrapper-in-cds-archive-heap' into jep-483-step-05-8293337-archive-method-handle-intrinsics
>>  - Merge branch 'jep-483-step-04-8293187-support-sun-invoke-util-wrapper-in-cds-archive-heap' into jep-483-step-05-8293337-archive-method-handle-intrinsics
>>  - some clean up
>>  - Merge branch 'jep-483-step-04-8293187-support-sun-invoke-util-wrapper-in-cds-archive-heap' into jep-483-step-05-8293337-archive-method-handle-intrinsics
>>  - Merge branch 'jep-483-step-04-8293187-support-sun-invoke-util-wrapper-in-cds-archive-heap' into jep-483-step-05-8293337-archive-method-handle-intrinsics
>>  - Merge branch 'jep-483-step-04-8293187-support-sun-invoke-util-wrapper-in-cds-archive-heap' into jep-483-step-05-8293337-archive-method-handle-intrinsics
>>  - Merge branch 'jep-483-step-04-8293187-support-sun-invoke-util-wrapper-in-cds-archive-heap' into jep-483-step-05-8293337-archive-method-handle-intrinsics
>>  - 8293337: Archive method handle intrinsics
>
> src/hotspot/share/classfile/systemDictionary.hpp line 349:
> 
>> 347:   // Second part of load_shared_class
>> 348:   static void load_shared_class_misc(InstanceKlass* ik, ClassLoaderData* loader_data) NOT_CDS_RETURN;
>> 349:   static void restore_archived_method_handle_intrinsics_impl(TRAPS);
> 
> Missing `NOT_CDS_RETURN` ?

Fixed.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20959#discussion_r1762100793


More information about the hotspot-dev mailing list