[SLPVectorizer] Swap the checking order of isCommutative and isConsecutiveAccess

NFC

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@258909 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Haicheng Wu 2016-01-27 04:59:05 +00:00
parent 903845e2b0
commit 2e1b6ea21c

View File

@ -1854,10 +1854,10 @@ void BoUpSLP::reorderAltShuffleOperands(ArrayRef<Value *> VL,
if (LoadInst *L1 = dyn_cast<LoadInst>(Right[j + 1])) { if (LoadInst *L1 = dyn_cast<LoadInst>(Right[j + 1])) {
Instruction *VL1 = cast<Instruction>(VL[j]); Instruction *VL1 = cast<Instruction>(VL[j]);
Instruction *VL2 = cast<Instruction>(VL[j + 1]); Instruction *VL2 = cast<Instruction>(VL[j + 1]);
if (isConsecutiveAccess(L, L1, DL, *SE) && VL1->isCommutative()) { if (VL1->isCommutative() && isConsecutiveAccess(L, L1, DL, *SE)) {
std::swap(Left[j], Right[j]); std::swap(Left[j], Right[j]);
continue; continue;
} else if (isConsecutiveAccess(L, L1, DL, *SE) && VL2->isCommutative()) { } else if (VL2->isCommutative() && isConsecutiveAccess(L, L1, DL, *SE)) {
std::swap(Left[j + 1], Right[j + 1]); std::swap(Left[j + 1], Right[j + 1]);
continue; continue;
} }
@ -1868,10 +1868,10 @@ void BoUpSLP::reorderAltShuffleOperands(ArrayRef<Value *> VL,
if (LoadInst *L1 = dyn_cast<LoadInst>(Left[j + 1])) { if (LoadInst *L1 = dyn_cast<LoadInst>(Left[j + 1])) {
Instruction *VL1 = cast<Instruction>(VL[j]); Instruction *VL1 = cast<Instruction>(VL[j]);
Instruction *VL2 = cast<Instruction>(VL[j + 1]); Instruction *VL2 = cast<Instruction>(VL[j + 1]);
if (isConsecutiveAccess(L, L1, DL, *SE) && VL1->isCommutative()) { if (VL1->isCommutative() && isConsecutiveAccess(L, L1, DL, *SE)) {
std::swap(Left[j], Right[j]); std::swap(Left[j], Right[j]);
continue; continue;
} else if (isConsecutiveAccess(L, L1, DL, *SE) && VL2->isCommutative()) { } else if (VL2->isCommutative() && isConsecutiveAccess(L, L1, DL, *SE)) {
std::swap(Left[j + 1], Right[j + 1]); std::swap(Left[j + 1], Right[j + 1]);
continue; continue;
} }