RFR: 8365262: [IR-Framework] Add simple way to add cross-product of flags [v6]
Emanuel Peter
epeter at openjdk.org
Fri Aug 22 14:10:54 UTC 2025
On Fri, 22 Aug 2025 09:36:45 GMT, Manuel Hässig <mhaessig at openjdk.org> wrote:
>> This PR adds the `TestFramework::addCrossProductScenarios` method to enable more ergonomic testing of the combination of all flag combinations. To illustrate its use, I also converted one test to use the new cross product functionality.
>>
>> Testing:
>> - [x] Github Actions
>> - [x] tier1,tier2 plus some internal testing on Oracle supported platforms
>
> Manuel Hässig has updated the pull request incrementally with one additional commit since the last revision:
>
> Improvements prompted by Emanuel
Changes requested by epeter (Reviewer).
test/hotspot/jtreg/compiler/lib/ir_framework/TestFramework.java line 387:
> 385: })
> 386: ),
> 387: (a, b) -> Stream.concat(a, b));
Wow, that's dense. Maybe a little comment could help here.
test/hotspot/jtreg/testlibrary_tests/ir_framework/tests/TestScenariosCrossProduct.java line 53:
> 51: TestFramework t = new TestFramework();
> 52: t.addCrossProductScenarios(null);
> 53: Asserts.fail("Should not have thrown exception");
Should or should not have thrown?
-------------
PR Review: https://git.openjdk.org/jdk/pull/26762#pullrequestreview-3144643027
PR Review Comment: https://git.openjdk.org/jdk/pull/26762#discussion_r2293833619
PR Review Comment: https://git.openjdk.org/jdk/pull/26762#discussion_r2293836677
More information about the hotspot-compiler-dev
mailing list