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