On Mon, 5 May 2025 23:50:41 GMT, Chen Liang <li...@openjdk.org> wrote:

>> Ioi Lam has updated the pull request with a new target base due to a merge 
>> or a rebase. The pull request now contains 20 commits:
>> 
>>  - Merge branch 'master' into 
>> 8354890-aot-init-methodhandleimpl-and-inner-classes
>>  - Comments from @liach and @ExE-Boss
>>  - Added more test case to increase coverage on possible core-lib usage 
>> patterns for MethodHandles
>>  - Merge branch 'master' into 
>> 8354890-aot-init-methodhandleimpl-and-inner-classes
>>  - 8354890: AOT-initialize j.l.i.MethodHandleImpl and inner classes
>>  - @fisk comment -- use proper HeapAccess to load referent; Also refactor 
>> AOTReferenceObjSupport::is_enabled()
>>  - Merge branch 'master' into 8354897-soft-weak-references-in-aot-cache
>>  - @fisk offline comments -- tighten up and simplify eligibility check; 
>> @DanHeidinga comment -- renamed to MethodType::assemblySetup()
>>  - @DanHeidinga comments
>>  - @fisk comment
>>  - ... and 10 more: https://git.openjdk.org/jdk/compare/9a0e6f33...0571ddc3
>
> The Java code change and the BSM coverage looks good to me. Requiring another 
> reviewer for hotspot changes.

Thanks @liach @iwanowww for the review

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

PR Comment: https://git.openjdk.org/jdk/pull/24956#issuecomment-2865231404

Reply via email to