RFR: 8290731: Clean up CDS handling of LambdaForm Species classes
Calvin Cheung
ccheung at openjdk.org
Thu Jul 21 05:16:22 UTC 2022
On Wed, 20 Jul 2022 20:53:17 GMT, Ioi Lam <iklam at openjdk.org> wrote:
> Please see the JBS issue for the reasons for this small clean up.
>
> The main change is in `LambdaFormInvokers::regenerate_holder_classes()` -- do not regenerate the Species_XXX classes. However, mark them as "generated" to be compatible with JVMTI.
>
> I renamed `Klass::is_regenerated()` to `Klass::is_generated_shared_class()` for clarity.
>
> I also updated the test case to be clear what it's trying to do -- Species_IL should be loaded from base archive and Species_JL should be loaded from the dynamic archive.
Looks good. One small nit.
test/hotspot/jtreg/runtime/cds/appcds/dynamicArchive/CDSLambdaInvoker.java line 32:
> 30: public static void main(String args[]) throws Throwable {
> 31: // The following calls trigger the generation of new Species classes
> 32: // that are not included in the base archive (or the default modules image.
Extra open parenthesis?
-------------
Marked as reviewed by ccheung (Reviewer).
PR: https://git.openjdk.org/jdk/pull/9577
More information about the hotspot-runtime-dev
mailing list