[PATCH] D157698: [AArch64][SME] Non-streaming compatible SCVTF emitted with --force-streaming-compatible-sve

Sander de Smalen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 14 03:12:51 PDT 2023


sdesmalen added inline comments.


================
Comment at: llvm/test/CodeGen/AArch64/sve-streaming-mode-fixed-length-int-to-fp.ll:1126
+
+define half @scvtf_v1i16_f16(ptr %0) {
+; CHECK-LABEL: scvtf_v1i16_f16:
----------------
Sorry, just spotted this after accepting the patch. The name `v1i16` seems wrong, because it's just converting an i16 -> f16, not <1 x i16> -> f16. I think you can just remove the `v1` from the name.

(same comment for all the other tests below).




CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D157698/new/

https://reviews.llvm.org/D157698



More information about the llvm-commits mailing list