LoopVectorizer: When checking for vectorizable types, also check

the StoreInst operands.

PR14705.



git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@171023 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Nadav Rotem 2012-12-24 09:14:18 +00:00
parent daf7b5c8f2
commit 9e5329d77e
2 changed files with 37 additions and 1 deletions

View File

@ -1464,13 +1464,20 @@ bool LoopVectorizationLegality::canVectorizeInstrs() {
return false;
}
// We do not re-vectorize vectors.
// Check that the instruction return type is vectorizable.
if (!VectorType::isValidElementType(it->getType()) &&
!it->getType()->isVoidTy()) {
DEBUG(dbgs() << "LV: Found unvectorizable type." << "\n");
return false;
}
// Check that the stored type is vectorizable.
if (StoreInst *ST = dyn_cast<StoreInst>(it)) {
Type *T = ST->getValueOperand()->getType();
if (!VectorType::isValidElementType(T))
return false;
}
// Reduction instructions are allowed to have exit users.
// All other instructions must not have external users.
if (!AllowedExit.count(it))

View File

@ -0,0 +1,29 @@
; RUN: opt < %s -loop-vectorize -mtriple=x86_64-unknown-linux-gnu -S
; Makre sure we are not crashing on this one.
target datalayout =
"e-p:64:64:64-i1:8:8-i8:8:8-i16:16:16-i32:32:32-i64:64:64-f32:32:32-f64:64:64-v64:64:64-v128:128:128-a0:0:64-s0:64:64-f80:128:128-n8:16:32:64-S128"
target triple = "x86_64-unknown-linux-gnu"
@glbl = external global [16 x { i64, i64 }], align 16
declare void @fn()
define void @test() {
entry:
br label %loop
loop:
%indvars.iv = phi i64 [ %indvars.iv.next, %loop ], [ 0, %entry ]
%tmp = getelementptr inbounds [16 x { i64, i64 }]* @glbl, i64 0, i64
%indvars.iv
store { i64, i64 } { i64 ptrtoint (void ()* @fn to i64), i64 0 }, { i64, i64 }* %tmp, align 16
%indvars.iv.next = add i64 %indvars.iv, 1
%lftr.wideiv = trunc i64 %indvars.iv.next to i32
%exitcond = icmp ne i32 %lftr.wideiv, 16
br i1 %exitcond, label %loop, label %exit
exit:
ret void
}