On Mon, 22 Sep 2025 09:38:16 GMT, Andrew Dinn <[email protected]> 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
