RFR: 8329077: C2 SuperWord: Add MoveD2L, MoveL2D, MoveF2I, MoveI2F [v3]
Galder Zamarreño
galder at openjdk.org
Wed Aug 20 12:01:38 UTC 2025
On Tue, 12 Aug 2025 08:22:11 GMT, Bhavana Kilambi <bkilambi at 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
More information about the core-libs-dev
mailing list