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