From d2e8c4cdea7ba95ddc603be923f56da8176c506a Mon Sep 17 00:00:00 2001 From: Nadav Rotem Date: Tue, 16 Jul 2013 22:57:06 +0000 Subject: [PATCH] flip the scev minus direction to simplify the code. llvm-svn: 186466 --- llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp b/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp index c1e8111..572f343 100644 --- a/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp +++ b/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp @@ -1004,7 +1004,7 @@ bool BoUpSLP::isConsecutiveAccess(Value *A, Value *B) { // Calculate the distance. const SCEV *PtrSCEVA = SE->getSCEV(PtrA); const SCEV *PtrSCEVB = SE->getSCEV(PtrB); - const SCEV *OffsetSCEV = SE->getMinusSCEV(PtrSCEVA, PtrSCEVB); + const SCEV *OffsetSCEV = SE->getMinusSCEV(PtrSCEVB, PtrSCEVA); const SCEVConstant *ConstOffSCEV = dyn_cast(OffsetSCEV); // Non constant distance. @@ -1013,10 +1013,10 @@ bool BoUpSLP::isConsecutiveAccess(Value *A, Value *B) { int64_t Offset = ConstOffSCEV->getValue()->getSExtValue(); Type *Ty = cast(PtrA->getType())->getElementType(); - // The Instructions are connsecutive if the size of the first load/store is + // The Instructions are consecutive if the size of the first load/store is // the same as the offset. int64_t Sz = DL->getTypeStoreSize(Ty); - return ((-Offset) == Sz); + return (Offset == Sz); } Value *BoUpSLP::getSinkBarrier(Instruction *Src, Instruction *Dst) { -- 2.7.4