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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to