RFR: 8368205: [TESTBUG] VectorMaskCompareNotTest.java crashes when MaxVectorSize=8 [v2]

[erifan]

On Wed, 15 Oct 2025 12:38:27 GMT, Martin Doerr <mdoerr at openjdk.org> wrote:

>> The test `VectorMaskCompareNotTest` requires 16 Byte vectors (or larger). If the machine only uses 8 Byte vectors, we get an exception in the static initializer because the code tries to use a 4 Byte vector which is unsupported (stack trace: see JBS issue [JDK-8369511](https://bugs.openjdk.org/browse/JDK-8369511)). This is an alternative to https://github.com/openjdk/jdk/pull/27749.
>
> Martin Doerr has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Add check if flag is available.

test/hotspot/jtreg/compiler/vectorapi/VectorMaskCompareNotTest.java line 38:

> 36:  * @summary test combining vector not operation with compare
> 37:  * @modules jdk.incubator.vector
> 38:  * @requires vm.opt.final.MaxVectorSize == "null" | vm.opt.final.MaxVectorSize >= 16

Should be
Suggestion:

 * @requires vm.opt.final.MaxVectorSize == "null" & vm.opt.final.MaxVectorSize >= 16


or
Suggestion:

 * @requires  vm.compiler2.enabled & vm.opt.final.MaxVectorSize >= 16

?

Assume this test is run with another compiler (like Graal) that doesn't support the option `MaxVectorSize`, then `vm.opt.final.MaxVectorSize == "null"` holds. But this can't guarantee that the max vector size >= 16.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27805#discussion_r2434435929