RFR: 8308994: C2: Re-implement experimental post loop vectorization [v4]

Pengfei Li pli at openjdk.org
Wed Sep 27 22:33:03 UTC 2023


On Tue, 4 Jul 2023 02:40:51 GMT, Pengfei Li <pli at openjdk.org> wrote:

>> test/hotspot/jtreg/compiler/vectorization/runner/ArrayInvariantFillTest.java line 69:
>> 
>>> 67:     @Test
>>> 68:     @IR(applyIfCPUFeatureOr = {"asimd", "true", "sse2", "true"},
>>> 69:         applyIf = {"OptimizeFill", "false"},
>> 
>> This seems unrelated. Why did you have to add this?
>
> Will cleanup this in JDK-8309697.

This is already fixed in another patch. Will close this comment.

>> test/hotspot/jtreg/compiler/vectorization/runner/VectorizationTestRunner.java line 84:
>> 
>>> 82:         TestFramework irTest = new TestFramework(klass);
>>> 83:         // Add extra VM options to enable more auto-vectorization chances
>>> 84:         irTest.addFlags("-XX:-OptimizeFill");
>> 
>> Aha, you removed this too. Fair enough. But since the runner is currently requiring everything to be `flagless`, now I cannot actually force `-XX:-OptimizeFill` from the outside. And that means that potentially the tests are never actually run with `OptimizeFill` off, and we never actually can check the IR rules. We lose test coverage. That makes me a bit nervous.
>> 
>> Suggestion: if tests actually require the flag off to execute the IR rule, then we should have two scenarios, one where the flag is on, and one when it is off.
>
> Again, will cleanup this in JDK-8309697.

It's already cleaned up in another patch. Will close it.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/14581#discussion_r1338288996
PR Review Comment: https://git.openjdk.org/jdk/pull/14581#discussion_r1338289635


More information about the hotspot-compiler-dev mailing list