RFR: 8338017: Add AOT command-line flag aliases [v2]
David Holmes
dholmes at openjdk.org
Thu Aug 29 05:09:21 UTC 2024
On Thu, 29 Aug 2024 04:24:02 GMT, Ioi Lam <iklam at openjdk.org> wrote:
>> This is the 1st PR for [JEP 483: Ahead-of-Time Class Loading & Linking](https://bugs.openjdk.org/browse/JDK-8315737).
>>
>> Add the following command-line options as specified in JEP 483:
>>
>>
>> - `-XX:AOTMode=off/record/create/auto/on`
>> - `-XX:AOTConfiguration=<file>.aotconfig`
>> - `-XX:AOTCache=<file>.aot`
>>
>> These options are implemented as aliases to existing command-line flags such as `-Xshare:dump`, `-XX:SharedArchiveFile`, `-XX:DumpLoadedClassesList`, etc.
>>
>> Please see the CSR (TODO) for detailed specification.
>>
>> -----
>> See [here](https://bugs.openjdk.org/browse/JDK-8315737) for the sequence of dependent RFEs for implementing JEP 483.
>
> 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 five additional commits since the last revision:
>
> - Fixed copyright dates
> - Merge branch 'master' of https://github.com/openjdk/jdk into jep-483-step-01-8338017-add-aot-command-line-aliases
> - Merge branch 'master' into jep-483-step-01-8338017-add-aot-command-line-aliases
> - Fixed whitespaces
> - 8338017: Add AOT command-line flag aliases
Seems reasonable but one issue flagged below.
Thanks.
src/java.base/share/native/libjli/java.c line 1521:
> 1519: dumpSharedSpaces = JNI_TRUE;
> 1520: }
> 1521: if (JLI_StrCmp(arg, "-XX:AOTMode=create") == 0) {
This is inappropriate - the launcher does not, and should not, process hotspot -XX options. Any aliasing should happen in the hotspot argument processing logic.
-------------
Changes requested by dholmes (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/20516#pullrequestreview-2267689489
PR Review Comment: https://git.openjdk.org/jdk/pull/20516#discussion_r1735573603
More information about the core-libs-dev
mailing list