RFR: 8300829: Make CtwRunner available as an independent tool [v2]
Xin Liu
xliu at openjdk.org
Thu Apr 6 02:48:14 UTC 2023
On Wed, 5 Apr 2023 17:01:10 GMT, Joshua Cao <duke at openjdk.org> wrote:
>> 1. Create an independent jar file with CtwRunner as the main class to make it easier to run
>> 2. Output the class files directly into the destination directory. Currently, CTW expects a `wb.jar`, but the jtreg tests that use CTWRunner has class files outside of a jar.
>> 3. Introduce `sun.hotspot.tools.ctwrunner.ctw_extra_args` option to pass extra arguments to CTW. Arguments are comma separated because working with spaces in bash can be kind of awkward, but I'm open to changing this part.
>>
>> ### Motivation
>> CTWRunner is a wrapper around CTW that will continue compiling beyond failure. It can be useful for testing compilation with certain flags. For example, one could run
>>
>>
>> JAVA_OPTIONS="-Dsun.hotspot.tools.ctwrunner.ctw_extra_args=-XX:+StressLCM,-XX:+StressGCM" ./ctwrunner.sh modules:java.base
>>
>>
>> To test compiling the java.base module with `-XX:+StressLCM -XX:+StressGCM`. This is advantageous over uses CTW because we can see the full list of crashes for the entire module.
>
> Joshua Cao has updated the pull request incrementally with two additional commits since the last revision:
>
> - Upgrade CTWRunner.java copyright header
> - Remove ctwrunner.jar and default value for CTW extra args
LGTM. I am not a reviewer. need other reviewers to approve this.
-------------
Marked as reviewed by xliu (Committer).
PR Review: https://git.openjdk.org/jdk/pull/13344#pullrequestreview-1373977066
More information about the hotspot-compiler-dev
mailing list