RFR: 8349953: Avoid editing AOTConfiguration file in "make test JTREG_AOT_JDK=true" [v2]
Ioi Lam
iklam at openjdk.org
Fri Feb 14 05:27:09 UTC 2025
> When running HotSpot jtreg tests in the "AOT mode", for example:
>
>
> make test JTREG_AOT_JDK=true open/test/hotspot/jtreg/runtime/stringtable
>
>
> Before this PR, in the test set up phase, we record several AOT configuration files by running a few separate Java tools (javac, javap, jlink, and jar), and then combine them together with sed, grep, sort and uniq:
>
> https://github.com/openjdk/jdk/blob/adc3f53d2403cd414a91e71c079b4108b2346da0/make/RunTests.gmk#L723-L744
>
> After [JDK-8348426](https://bugs.openjdk.org/browse/JDK-8348426), the AOT configuration file will change to a binary format and can no longer be edited this way. In preparation for [JDK-8348426](https://bugs.openjdk.org/browse/JDK-8348426), we should change the "JTREG_AOT_JDK=true" set up to run a single Java program that accomplishes the same effect as the current implementation.
>
> ** Changes in this PR **
>
> This PR combines the invocation of these Java tools into a single Java program, so we just have a single AOT configuration file. It also uses the `-XX:ExtraSharedClassListFile` option to include the default classlist from the JDK home directory,
Ioi Lam has updated the pull request incrementally with one additional commit since the last revision:
Comments from @erikj79
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/23620/files
- new: https://git.openjdk.org/jdk/pull/23620/files/0a645b69..c1aaf635
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=23620&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=23620&range=00-01
Stats: 224 lines in 5 files changed: 142 ins; 62 del; 20 mod
Patch: https://git.openjdk.org/jdk/pull/23620.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/23620/head:pull/23620
PR: https://git.openjdk.org/jdk/pull/23620
More information about the build-dev
mailing list