RFR: 8365262: [IR-Framework] Add simple way to add cross-product of flags

Benoît Maillard bmaillard at openjdk.org
Mon Aug 18 08:07:11 UTC 2025


On Wed, 13 Aug 2025 14:38:01 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

Looks good to me, I only have one minor suggestion.

test/hotspot/jtreg/compiler/lib/ir_framework/TestFramework.java line 368:

> 366:                 .flatMap(setElement -> crossProductHelper(idx + 1, sets)
> 367:                         .map(set -> {
> 368:                             Set<String> newSet = new HashSet(set);

Suggestion:

                            Set<String> newSet = new HashSet<>(set);


You should use the diamond operator here to use the generic type instead of the raw `HashSet` type

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

PR Review: https://git.openjdk.org/jdk/pull/26762#pullrequestreview-3127232286
PR Review Comment: https://git.openjdk.org/jdk/pull/26762#discussion_r2281603073


More information about the hotspot-compiler-dev mailing list