RFR: 8338126 : C2 SuperWord: VectorCastF2HF / vcvtps2ph produces wrong results for vector length 2 [v3]

Sandhya Viswanathan sviswanathan at openjdk.org
Wed Oct 16 01:39:50 UTC 2024


> When Float.floatToFloat16 is vectorized using a 2-element vector width due to dependencies, we incorrectly generate a 4-element vcvtps2ph with memory as destination storing 8 bytes instead of desired 4 bytes.  This issue is fixed in this PR by limiting the memory version of match rule to 4-element vector and above.
> Also a regression test case is added accordingly.
> 
> Best Regards,
> Sandhya

Sandhya Viswanathan has updated the pull request incrementally with one additional commit since the last revision:

  Run test on all platforms

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/21480/files
  - new: https://git.openjdk.org/jdk/pull/21480/files/ed299327..f2981374

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=21480&range=02
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=21480&range=01-02

  Stats: 11 lines in 2 files changed: 5 ins; 3 del; 3 mod
  Patch: https://git.openjdk.org/jdk/pull/21480.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/21480/head:pull/21480

PR: https://git.openjdk.org/jdk/pull/21480


More information about the hotspot-compiler-dev mailing list