Integrated: 8280767: -XX:ArchiveClassesAtExit does not archive BoundMethodHandle$Species classes
Yumin Qi
minqi at openjdk.java.net
Fri Feb 4 19:23:14 UTC 2022
On Wed, 2 Feb 2022 23:26:32 GMT, Yumin Qi <minqi at openjdk.org> wrote:
> Hi, Please review
>
> When do dynamic dump, the call to LambdaForInvolkers::append_filtered for generating lambda holder classes will only allow the result which matches the filtering list to be added . This filters out the SPECIES method handle holder classes, and they won't be archived in dynamic shared archive. The patch is doing followings:
> 1) Removed the call to LambdaForInvolkers::append_filtered, instead just call LambdaForInvolkers::append so the SPECIES can be added to the regeneration list.
> 2) Added an enum for shared flag in Klass, _generated to indicate the class is regenerated and it should not be loaded from CDS if CFLH is enabled. If the class is a SPECIES and also loaded from base archive, skip regenerating it for dynamic archive.
> 3) If CFLH is enabled, for regenerated classes, avoid calling CFLH.
>
> Tests: tier1,tier4
>
> Thanks
> Yumin
This pull request has now been integrated.
Changeset: 8e4ef818
Author: Yumin Qi <minqi at openjdk.org>
URL: https://git.openjdk.java.net/jdk/commit/8e4ef818a90de35ae75e7f82a780653d623bb29c
Stats: 174 lines in 7 files changed: 159 ins; 7 del; 8 mod
8280767: -XX:ArchiveClassesAtExit does not archive BoundMethodHandle$Species classes
Reviewed-by: iklam, ccheung
-------------
PR: https://git.openjdk.java.net/jdk/pull/7329
More information about the hotspot-runtime-dev
mailing list