RFR: 8280767: -XX:ArchiveClassesAtExit does not archive BoundMethodHandle$Species classes [v4]

Yumin Qi minqi at openjdk.java.net
Fri Feb 4 18:39:48 UTC 2022


> 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

Yumin Qi has updated the pull request incrementally with one additional commit since the last revision:

  Add comment to test

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/7329/files
  - new: https://git.openjdk.java.net/jdk/pull/7329/files/1ef457d6..d5115e4d

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7329&range=03
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7329&range=02-03

  Stats: 1 line in 1 file changed: 1 ins; 0 del; 0 mod
  Patch: https://git.openjdk.java.net/jdk/pull/7329.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/7329/head:pull/7329

PR: https://git.openjdk.java.net/jdk/pull/7329


More information about the hotspot-runtime-dev mailing list