RFR: 8368174: Proactive initialization of @AOTSafeClassInitializer classes [v2]
Ioi Lam
iklam at openjdk.org
Wed Sep 24 21:01:57 UTC 2025
On Mon, 22 Sep 2025 09:38:16 GMT, Andrew Dinn <adinn at openjdk.org> wrote:
>> Ioi Lam has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains six additional commits since the last revision:
>>
>> - Exclude new test from hotspot_aot_classlinking and hotspot_appcds_dynamic test groups
>> - @adinn and @liach comments
>> - Merge branch 'master' into 8368174-proactive-init-aot-safe-class-initializer
>> - updated javadoc for AOTSafeClassInitializer.java
>> - Fix
>> - imported
>
> This is much simpler and clearer than the previous version making it easier for non-Leyden devs to understand what they are buying into when they use the annotation. Nice work @ashu-mehra and @iklam.
> I made a few suggestions to clarify comments which you are free to adopt or drop as you see fit. Otherwise looks good to go.
Thanks @adinn @liach @ashu-mehra for the review
-------------
PR Comment: https://git.openjdk.org/jdk/pull/27402#issuecomment-3330687291
More information about the core-libs-dev
mailing list