[PATCH] D18324: [SLP] Remove unnecessary member variables by using container APIs.
Chad Rosier via llvm-commits
llvm-commits at lists.llvm.org
Mon Mar 21 12:52:52 PDT 2016
This revision was automatically updated to reflect the committed changes.
Closed by commit rL263975: [SLP] Remove unnecessary member variables by using container APIs. (authored by mcrosier).
Changed prior to commit:
http://reviews.llvm.org/D18324?vs=51216&id=51217#toc
Repository:
rL LLVM
http://reviews.llvm.org/D18324
Files:
llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp
Index: llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp
===================================================================
--- llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp
+++ llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp
@@ -3439,8 +3439,9 @@
collectSeedInstructions(BB);
// Vectorize trees that end at stores.
- if (NumStores > 0) {
- DEBUG(dbgs() << "SLP: Found " << NumStores << " stores.\n");
+ if (!Stores.empty()) {
+ DEBUG(dbgs() << "SLP: Found stores for " << Stores.size()
+ << " underlying objects.\n");
Changed |= vectorizeStoreChains(R);
}
@@ -3450,8 +3451,9 @@
// Vectorize the index computations of getelementptr instructions. This
// is primarily intended to catch gather-like idioms ending at
// non-consecutive loads.
- if (NumGEPs > 0) {
- DEBUG(dbgs() << "SLP: Found " << NumGEPs << " GEPs.\n");
+ if (!GEPs.empty()) {
+ DEBUG(dbgs() << "SLP: Found GEPs for " << GEPs.size()
+ << " underlying objects.\n");
Changed |= vectorizeGEPIndices(BB, R);
}
}
@@ -3527,12 +3529,6 @@
/// The getelementptr instructions in a basic block organized by base pointer.
WeakVHListMap GEPs;
- /// The number of store instructions in a basic block.
- unsigned NumStores;
-
- /// The number of getelementptr instructions in a basic block.
- unsigned NumGEPs;
-
unsigned MaxVecRegSize; // This is set by TTI or overridden by cl::opt.
unsigned MinVecRegSize; // Set by cl::opt (default: 128).
};
@@ -3670,7 +3666,6 @@
// Initialize the collections. We will make a single pass over the block.
Stores.clear();
GEPs.clear();
- NumStores = NumGEPs = 0;
// Visit the store and getelementptr instructions in BB and organize them in
// Stores and GEPs according to the underlying objects of their pointer
@@ -3685,7 +3680,6 @@
if (!isValidElementType(SI->getValueOperand()->getType()))
continue;
Stores[GetUnderlyingObject(SI->getPointerOperand(), *DL)].push_back(SI);
- ++NumStores;
}
// Ignore getelementptr instructions that have more than one index, a
@@ -3698,7 +3692,6 @@
if (!isValidElementType(Idx->getType()))
continue;
GEPs[GetUnderlyingObject(GEP->getPointerOperand(), *DL)].push_back(GEP);
- ++NumGEPs;
}
}
}
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D18324.51217.patch
Type: text/x-patch
Size: 2412 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160321/bef05d8f/attachment.bin>
More information about the llvm-commits
mailing list