RFR: 8369423: Reduce execution time of testlibrary_tests/ir_framework/tests/TestDFlags.java

[Tobias Hartmann]

On Wed, 8 Oct 2025 14:58:06 GMT, Damon Fenacci <dfenacci at openjdk.org> wrote:

>> The test `testlibrary_tests/ir_framework/tests/TestDFlags.java` runs 11 separate IR framework runs by enabling only a single property/D flag in each run. This seems like a waste of resources for just a sanity run (we don't do any additional verification). Moreover, some newer property flags are missing.
>> 
>> I suggest to cut it down to 2 runs: 
>> - IR verification enabled (i.e. -DVerifyIR=true, which is the default) 
>> - IR verification disabled (i.e. -DVerifyIR=false)
>> 
>> In both runs we simultaneously set all property flags to some non-default value as a sanity test.
>> 
>> This reduces the test execution time from around 20-30s down to 3-4s.
>> 
>> Thanks,
>> Christian
>
> test/hotspot/jtreg/compiler/lib/ir_framework/README.md line 178:
> 
>> 176: - `-DWarmup=200`: Provide a new default value of the number of warm-up iterations (framework default is 2000). This might have an influence on the resulting IR and could lead to matching failures (the user can also set a fixed default warm-up value in a test with `testFrameworkObject.setDefaultWarmup(200)`).
>> 177: - `-DReportStdout=true`: Print the standard output of the test VM.
>> 178: - `-DVerbose=true`: Enable more fain-grained logging (slows the execution down).
> 
> Suggestion:
> 
> - `-DVerbose=true`: Enable more fine-grained logging (slows the execution down).
> 
> 
> it just caught my eye 😉

Ouh, good catch!

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27690#discussion_r2415607037