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