On Tue, 3 Mar 2026 09:55:03 GMT, Jatin Bhateja <[email protected]> wrote:
> Strict assertion added to jdk.incubator.vector.CPUFeatures does not take into > account MaxVectorSize settings. As a result we observer serval VectorAPI > VectorMathLibrary test failures when MaxVectorSize is smaller than vector > size supported by target. > > Patch removes these assertions, all vector API tests are passing with > smaller MaxVectorSize with assertions enabled (-ea -es). > > Kindly review and share your feedback. > > Best Regards, > Jatin src/jdk.incubator.vector/share/classes/jdk/incubator/vector/CPUFeatures.java line 79: > 77: assert SUPPORTS_AVX512F == > (VectorShape.getMaxVectorBitSize(int.class) == 512); > 78: assert SUPPORTS_AVX2 == > (VectorShape.getMaxVectorBitSize(byte.class) >= 256); > 79: assert SUPPORTS_AVX == > (VectorShape.getMaxVectorBitSize(float.class) >= 256); So is there any issue for the `suffix` if `MaxVectorSize` is lower than the hardware support max vector size? https://github.com/openjdk/jdk/blob/921da0a9734d9fae2e7b0e129d2cc6949ad0b5a6/src/jdk.incubator.vector/share/classes/jdk/incubator/vector/VectorMathLibrary.java#L123-L136 ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/30015#discussion_r2943977791
