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

[Christian Hagedorn]

On Wed, 8 Oct 2025 11:30:43 GMT, Christian Hagedorn <chagedorn 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

This pull request has now been integrated.

Changeset: dd410e0b
Author:    Christian Hagedorn <chagedorn at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/dd410e0b776a01b617a457786b11ddf87d3b4d60
Stats:     22 lines in 2 files changed: 7 ins; 3 del; 12 mod

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

Reviewed-by: thartmann, dfenacci

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

PR: https://git.openjdk.org/jdk/pull/27690