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