[PATCH] D101834: [llvm][sve] Lowering for VLS MLOAD/MSTORE

Sander de Smalen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 20 03:40:30 PDT 2021


sdesmalen accepted this revision.
sdesmalen added a comment.
This revision is now accepted and ready to land.

LGTM, thanks for adding the test.



================
Comment at: llvm/test/CodeGen/AArch64/sve-fixed-length-masked-loads.ll:34-35
+; CHECK-NEXT: umov [[W1:w[0-9]+]], v[[N2]].h[1]
+; CHECK-NEXT: fmov [[S0:s[0-9]+]], [[W0]]
+; CHECK-NEXT: mov [[V0:v[0-9]+]].s[1], [[W1]]
+; CHECK-NEXT: shl [[V0]].2s, [[V0]].2s, #16
----------------
nit:
  fmov s[[V0:[0-9]+]], [[W0]]
  mov v[[V0]].s[1], [[W1]]


================
Comment at: llvm/test/CodeGen/AArch64/sve-fixed-length-masked-loads.ll:42
+; CHECK-NEXT: mov [[V1:v[0-9]+]].h[0], [[W1]]
+; CHECK-NEXT: mov [[V1:v[0-9]+]].h[1], [[W0]]
+; CHECK-NEXT: shl [[V0]].4h, [[V1]].4h, #15
----------------
nit:
  mov [[V1]].h[1], [[W0]]


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101834



More information about the llvm-commits mailing list