RFR: 8367387: Add @AOTInitialize annotation [v2]

Ioi Lam iklam at openjdk.org
Tue Sep 16 17:50:12 UTC 2025


On Tue, 16 Sep 2025 09:25:21 GMT, Andrew Dinn <adinn at openjdk.org> wrote:

>> Ioi Lam has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Exclude more GC heap size tests as AOT cache size has increased for "make test JTREG=AOT_JDK=onestep ..."
>
> src/hotspot/share/cds/aotMetaspace.cpp line 806:
> 
>> 804:   AOTClassInitializer::init_test_class(CHECK);
>> 805: 
>> 806:   if (CDSConfig::is_dumping_final_static_archive()) {
> 
> What is the reason for moving this earlier to before link_all_loaded_classes?

I added comment about the order of this code vs link_all_loaded_classes().

Before switching the order, I found that in some rare cases classes were removed from the AOT cache because they are unlinked after `FinalImageRecipes::apply_recipes()`.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27024#discussion_r2353226013


More information about the core-libs-dev mailing list