From 30dfbf03a206ad4f2a18d7fb904831ee728b6cbc Mon Sep 17 00:00:00 2001 From: David Sherwood Date: Thu, 28 May 2020 14:22:49 +0100 Subject: [PATCH] [CodeGen,AArch64] Fix up warnings in splitStores The code for trying to split up stores is designed for NEON vectors, where we support arbitrary alignments. It's an optimisation designed to improve performance by using smaller, aligned stores. However, we currently only support 16 byte alignments for SVE vectors anyway so we may as well bail out early. This change fixes up remaining warnings in a couple of tests: CodeGen/AArch64/sve-callbyref-notailcall.ll CodeGen/AArch64/sve-calling-convention-byref.ll Differential Revision: https://reviews.llvm.org/D80720 --- llvm/lib/Target/AArch64/AArch64ISelLowering.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/llvm/lib/Target/AArch64/AArch64ISelLowering.cpp b/llvm/lib/Target/AArch64/AArch64ISelLowering.cpp index e7f1c59..43a85d5 100644 --- a/llvm/lib/Target/AArch64/AArch64ISelLowering.cpp +++ b/llvm/lib/Target/AArch64/AArch64ISelLowering.cpp @@ -12138,7 +12138,8 @@ static SDValue splitStores(SDNode *N, TargetLowering::DAGCombinerInfo &DCI, SDValue StVal = S->getValue(); EVT VT = StVal.getValueType(); - if (!VT.isVector()) + + if (!VT.isFixedLengthVector()) return SDValue(); // If we get a splat of zeros, convert this vector store to a store of -- 2.7.4