RFR: 8317121: vector_masked_load instruction is moved too early after JDK-8286941

[Vladimir Kozlov]

On Tue, 3 Oct 2023 00:29:10 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>> test/hotspot/jtreg/compiler/vectorization/TestMaskedVectors.java line 31:
>> 
>>> 29:  * @modules java.base/jdk.internal.misc
>>> 30:  * @library /test/lib /
>>> 31:  * @run main/othervm -XX:+IgnoreUnrecognizedVMOptions -XX:-TieredCompilation -Xbatch -XX:CompileCommand=quiet -XX:CompileCommand=compileonly,TestMaskedVectors::test* -XX:+StressLCM -XX:+StressGCM -XX:StressSeed=2210259638 TestMaskedVectors
>> 
>> Does this need to enable diagnostic VM options the `Stress*` flags?
>> 
>> https://github.com/openjdk/jdk/blob/6e1aacdfba5a32f7b071eea8039888d275827e83/src/hotspot/share/opto/c2_globals.hpp#L44-L60
>> 
>> Suggestion:
>> 
>>  * @run main/othervm -XX:+IgnoreUnrecognizedVMOptions -XX:-TieredCompilation -Xbatch -XX:CompileCommand=quiet -XX:CompileCommand=compileonly,TestMaskedVectors::test* -XX:+UnlockDiagnosticVMOptions -XX:+StressLCM -XX:+StressGCM -XX:StressSeed=2210259638 TestMaskedVectors
>
> Yes, I have it in additional changes which are coming. I am waiting for our internal tests to finish.

And that is why this is still DRAFT ;^)

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16015#discussion_r1343281498