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

Manuel Hässig mhaessig at openjdk.org
Tue Aug 26 13:24:00 UTC 2025


On Mon, 25 Aug 2025 15:16:41 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> Manuel Hässig has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 15 additional commits since the last revision:
>> 
>>  - Merge branch 'master' into JDK-8365262
>>  - Remove excess newline
>>  - Fix indentation
>>  - Improve comments
>>  - Fix copy pasta mistakes
>>  - Improvements prompted by Emanuel
>>  - Fix test
>>  - Better counting in tests
>>  - post processing of flags and documentation
>>  - Make the test work
>>  - ... and 5 more: https://git.openjdk.org/jdk/compare/c1fb5caa...771924f0
>
> test/hotspot/jtreg/compiler/lib/ir_framework/TestFramework.java line 380:
> 
>> 378:             .reduce(
>> 379:                 Stream.of(Collections.<String>emptyList()), // Initialize Stream<List<String>> acc with a Stream containing an empty list of Strings.
>> 380:                 (acc, set) ->  // (Stream<List<String>>, Stream<List<String>>) -> Stream<List<String>>
> 
> You could probably put the types in the argument capture, no? Then it would become actual code.

Good point. This made me realize, I wrote down the wrong types...

> test/hotspot/jtreg/compiler/lib/ir_framework/TestFramework.java line 397:
> 
>> 395:                      .flatMap(Collection::stream) // Flatten the Stream<List<String>> into Stream<String>>.
>> 396:                      .filter(s -> !s.isEmpty()) // Remove empty string flags.
>> 397:                      .distinct()
> 
> Is this necessary? Could this reorder things? Sometimes order is relevant.

It is not. I removed it.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26762#discussion_r2300966957
PR Review Comment: https://git.openjdk.org/jdk/pull/26762#discussion_r2300967457


More information about the hotspot-compiler-dev mailing list