[PATCH] D69791: [ARM,MVE] Add intrinsics for gather/scatter load/stores.

[Dave Green via Phabricator via cfe-commits]

dmgreen added a comment.

Looks good as far as I can see.



================
Comment at: clang/include/clang/Basic/arm_mve.td:78
+    def _gather_base: Intrinsic<
+      Vector, (args VecOf<Unsigned<Scalar>>:$addr, imm_mem7bit<size>:$offset),
+      (IRInt<"vldr_gather_base", [Vector, VecOf<Unsigned<Scalar>>]> $addr, $offset)>;
----------------
Is it worth giving a name to VecOf<Unsigned<Scalar>>?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69791