[llvm] r257500 - rangify; NFCI (original) (raw)

Sanjay Patel via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 12 10:47:59 PST 2016

Previous message: [llvm] r257499 - Auto-link with ole32.dll to simplify building LLVM.dll
Next message: [PATCH] D16119: [CMake] Adding experimental support for tvOS and watchOS
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

Author: spatel Date: Tue Jan 12 12:47:59 2016 New Revision: 257500

URL: http://llvm.org/viewvc/llvm-project?rev=257500&view=rev Log: rangify; NFCI

Modified: llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp

Modified: llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp?rev=257500&r1=257499&r2=257500&view=diff

--- llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp (original) +++ llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp Tue Jan 12 12:47:59 2016 @@ -1782,30 +1782,29 @@ int BoUpSLP::getTreeCost() {

unsigned BundleWidth = VectorizableTree[0].Scalars.size();

for (unsigned i = 0, e = VectorizableTree.size(); i != e; ++i) {
int C = getEntryCost(&VectorizableTree[i]);

for (TreeEntry &TE : VectorizableTree) {
int C = getEntryCost(&TE); DEBUG(dbgs() << "SLP: Adding cost " << C << " for bundle that starts with "

     << *VectorizableTree[i].Scalars[0] << " .\n");

```
     << TE.Scalars[0] << " .\n");
```
Cost += C; }

SmallSet<Value *, 16> ExtractCostCalculated; int ExtractCost = 0;

for (UserList::iterator I = ExternalUses.begin(), E = ExternalUses.end();
```
  I != E; ++I) {
```

for (ExternalUser &EU : ExternalUses) { // We only add extract cost once for the same scalar.

if (!ExtractCostCalculated.insert(I->Scalar).second)

if (!ExtractCostCalculated.insert(EU.Scalar).second) continue;

// Uses by ephemeral values are free (because the ephemeral value will be // removed prior to code generation, and so the extraction will be // removed as well).

if (EphValues.count(I->User))

if (EphValues.count(EU.User)) continue;

VectorType *VecTy = VectorType::get(I->Scalar->getType(), BundleWidth);

VectorType *VecTy = VectorType::get(EU.Scalar->getType(), BundleWidth); ExtractCost += TTI->getVectorInstrCost(Instruction::ExtractElement, VecTy,

                                      I->Lane);

                                      EU.Lane);

}

Cost += getSpillCost();

@@ -4020,9 +4019,8 @@ bool SLPVectorizer::vectorizeChainsInBlo

 // Collect the incoming values from the PHIs.
 Incoming.clear();

for (BasicBlock::iterator instr = BB->begin(), ie = BB->end(); instr != ie;
```
    ++instr) {
```
```
 PHINode *P = dyn_cast<PHINode>(instr);
```

for (Instruction &I : *BB) {

 PHINode *P = dyn_cast<PHINode>(&I);
 if (!P)
   break;

Previous message: [llvm] r257499 - Auto-link with ole32.dll to simplify building LLVM.dll
Next message: [PATCH] D16119: [CMake] Adding experimental support for tvOS and watchOS
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list