nicolas/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[SLP] Allow vectorization of the instruction from the same basic blocks only, NFC.

Browse Source 
Summary:
After some changes in SLP vectorizer we missed some additional checks to
limit the instructions for vectorization. We should not perform analysis
of the instructions if the parent of instruction is not the same as the
parent of the first instruction in the tree or it was analyzed already.

Subscribers: mzolotukhin

Differential Revision: https://reviews.llvm.org/D34881

llvm-svn: 309425
This commit is contained in:
Alexey Bataev
2017-07-28 20:11:16 +00:00
parent 74ddba0c95
commit e109655c90
1 changed files with 8 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
View File

@@ -4776,7 +4776,7 @@ static bool tryToVectorizeHorReductionOrInstOperands(
if (!Root)
return false;
if (Root->getParent() != BB)
if (Root->getParent() != BB || isa<PHINode>(Root))
return false;
// Start analysis starting from Root instruction. If horizontal reduction is
// found, try to vectorize it. If it is not a horizontal reduction or
@@ -4797,7 +4797,7 @@ static bool tryToVectorizeHorReductionOrInstOperands(
if (!V)
continue;
auto *Inst = dyn_cast<Instruction>(V);
if (!Inst || isa<PHINode>(Inst))
if (!Inst)
continue;
if (auto *BI = dyn_cast<BinaryOperator>(Inst)) {
HorizontalReduction HorRdx;
@@ -4831,9 +4831,14 @@ static bool tryToVectorizeHorReductionOrInstOperands(
}
// Try to vectorize operands.
// Continue analysis for the instruction from the same basic block only to
// save compile time.
if (++Level < RecursionMaxDepth)
for (auto *Op : Inst->operand_values())
Stack.emplace_back(Op, Level);
if (VisitedInstrs.insert(Op).second)
if (auto *I = dyn_cast<Instruction>(Op))
if (!isa<PHINode>(Inst) && I->getParent() == BB)
Stack.emplace_back(Op, Level);
}
return Res;
}