[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