From: Nadav Rotem Date: Tue, 16 Jul 2013 22:57:06 +0000 (+0000) Subject: flip the scev minus direction to simplify the code. X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=71cc5e4ea25c27d0b7d410a8ee7a83a19769fdcc;p=oota-llvm.git flip the scev minus direction to simplify the code. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@186466 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/Transforms/Vectorize/SLPVectorizer.cpp b/lib/Transforms/Vectorize/SLPVectorizer.cpp index c1e811126d6..572f3436804 100644 --- a/lib/Transforms/Vectorize/SLPVectorizer.cpp +++ b/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) {