RFR: 8307468: CDS Lambda Proxy classes are regenerated in dynamic dump [v2]
Ashutosh Mehra
duke at openjdk.org
Wed Sep 6 13:05:39 UTC 2023
On Tue, 5 Sep 2023 17:13:24 GMT, Calvin Cheung <ccheung at openjdk.org> wrote:
>> Please review this changeset for fixing the following issues:
>>
>> 1. for lambda proxy classes already stored in the static archive, during CDS dynamic dump, those classes should not be regenerated and stored in the dynamic archive; (see changes in jvm.cpp)
>> 2. handle multiple lambda proxy classes with the same invoke dynamic info stored in both static and dynamic archives. (see changes in systemDictionaryShared.cpp)
>>
>> Also added a test.
>>
>> Passed tiers 1 - 4 testing.
>
> Calvin Cheung has updated the pull request incrementally with one additional commit since the last revision:
>
> review comments from @ashu-mehra and @turbanoff
Marked as reviewed by ashu-mehra at github.com (no known OpenJDK username).
@calvinccheung thanks for addressing the comments. lgtm.
-------------
PR Review: https://git.openjdk.org/jdk/pull/15524#pullrequestreview-1613286680
PR Comment: https://git.openjdk.org/jdk/pull/15524#issuecomment-1708314625
More information about the hotspot-runtime-dev
mailing list