[PATCH] D69788: [ARM MVE] Remove accidental 64-bit vst2/vld2 intrinsics.

Simon Tatham via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 4 01:54:04 PST 2019


simon_tatham created this revision.
simon_tatham added a reviewer: dmgreen.
Herald added subscribers: cfe-commits, kristof.beyls.
Herald added a project: clang.

ACLE defines no such intrinsic as vst2q_u64, and the MVE instruction
set has no corresponding instruction. But I had accidentally added
them to the fledgling <arm_mve.h> anyway, and if you used them, you'd
get a compiler crash.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D69788

Files:
  clang/include/clang/Basic/arm_mve.td


Index: clang/include/clang/Basic/arm_mve.td
===================================================================
--- clang/include/clang/Basic/arm_mve.td
+++ clang/include/clang/Basic/arm_mve.td
@@ -18,7 +18,7 @@
 
 include "arm_mve_defs.td"
 
-let params = T.All in
+let params = T.Usual in
 foreach n = [ 2, 4 ] in {
   def "vst"#n#"q": Intrinsic<Void, (args Ptr<Scalar>, MultiVector<n>),
                              (CustomCodegen<"VST24"> n:$NumVectors,


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D69788.227656.patch
Type: text/x-patch
Size: 459 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20191104/ae692999/attachment.bin>


More information about the cfe-commits mailing list