On Fri, 19 Sep 2025 16:57:42 GMT, Chen Liang <[email protected]> wrote:
>> Ioi Lam has updated the pull request with a new target base due to a merge >> or a rebase. The pull request now contains 23 commits: >> >> - Updated requirements for annotations on supertypes for AOTInitialize and >> AOTSafeClassInitializer >> - Merge branch 'master' into aot-initialize-math-utils >> - Added logging about @AOTSafeClassInitializer classes that have not been >> initialized >> - Added comment about the order of FinalImageRecipes::apply_recipes() vs >> link_all_loaded_classes() >> - Exclude more GC heap size tests as AOT cache size has increased for "make >> test JTREG=AOT_JDK=onestep ..." >> - Added javadoc; added test case; code clean up >> - Merge branch 'master' into aot-initialize-math-utils >> - cleanup >> - Merge branch 'master' into aot-initialize-math-utils >> - Merge branch '8366477-refactor-aot-related-flag-bits-in-klass-hpp' into >> aot-initialize-math-utils >> - ... and 13 more: https://git.openjdk.org/jdk/compare/7ec3fa5f...d21bfb12 > > src/hotspot/share/cds/aotMetaspace.cpp line 1266: > >> 1264: if (HAS_PENDING_EXCEPTION) { >> 1265: ResourceMark rm; >> 1266: log_error(aot, init)("class %s has @AOTForceInitialize but >> failed to initialize", > > Suggestion: > > log_error(aot, init)("class %s has @AOTInitialize but failed to > initialize", Thanks for catching this. Fixed. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/27024#discussion_r2363778305
