On Tue, 12 Aug 2025 08:22:11 GMT, Bhavana Kilambi <bkila...@openjdk.org> wrote:
>> Btw, I've noticed that `TestFloat16ScalarOperations` does not have `package` >> definition. Is that an oversight? It runs fine in spite of not having it > > Hi, as you mostly touched the auto-vectorization part of c2, could you please > run these float16 tests as well (most of these enable auto-vectorization for > Float16) - > > `compiler/vectorization/TestFloat16VectorOperations.java` > `compiler/vectorization/TestFloatConversionsVectorNaN.java` > `compiler/vectorization/TestFloatConversionsVector.java` > `compiler/vectorization/TestFloat16ToFloatConv.java` > `compiler/vectorization/TestFloat16VectorConvChain.java` > `compiler/intrinsics/float16/*` @Bhavana-Kilambi I've run these tests: "test/hotspot/jtreg/compiler/c2/irTests/ConvF2HFIdealizationTests.java" "test/hotspot/jtreg/compiler/c2/irTests/TestFloat16ScalarOperations.java" "test/hotspot/jtreg/compiler/intrinsics/float16/*" "test/hotspot/jtreg/compiler/vectorization/TestFloat16ToFloatConv.java" "test/hotspot/jtreg/compiler/vectorization/TestFloat16VectorConvChain.java" "test/hotspot/jtreg/compiler/vectorization/TestFloat16VectorOperations.java" "test/hotspot/jtreg/compiler/vectorization/TestFloatConversionsVector.java" "test/hotspot/jtreg/compiler/vectorization/TestFloatConversionsVectorNaN.java" On x86: Test results: passed: 11; did not meet platform requirements: 1 (TestFloatConversionsVectorNaN is for riscv) On graviton 3 aarch64: Test results: passed: 10; failed: 1; did not meet platform requirements: 1 The failure on aarch64 is already existing issue [JDK-8361582](https://bugs.openjdk.org/browse/JDK-8361582) ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/26457#discussion_r2287939342
