RFR: 8329706: Implement -XX:+AOTClassLinking [v9]

Ioi Lam iklam at openjdk.org
Thu Sep 19 03:19:36 UTC 2024


On Wed, 18 Sep 2024 05:35:42 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> Ioi Lam has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fixed ZERO build
>
> test/hotspot/jtreg/runtime/cds/appcds/aotClassLinking/AOTClassLinkingVMOptions.java line 57:
> 
>> 55:         testCase("Archived full module graph must be enabled at runtime");
>> 56:         TestCommon.run("-cp", appJar, "-Djdk.module.validation=1", "Hello")
>> 57:             .assertAbnormalExit("CDS archive has aot-linked classes." +
> 
> Nit: align the dots

The CDS test cases indent by 4 spaces in this situation. I searched for `'^ *[.]'` lines in the JDK source code and indentation of 8 and 4 spaces seem to be most common.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20843#discussion_r1766089235


More information about the serviceability-dev mailing list