diff options
| author | Benjamin Kramer <benny.kra@googlemail.com> | 2013-05-17 14:48:17 +0000 | 
|---|---|---|
| committer | Benjamin Kramer <benny.kra@googlemail.com> | 2013-05-17 14:48:17 +0000 | 
| commit | d84a63398e7cb72816f3e2189892af705340bcd8 (patch) | |
| tree | f91ee0bbf19e2316a790c4edcd7f2b0f110f206b /llvm/lib/Transforms | |
| parent | 84cf70ec09c7c9e890fbe71823f6f1b64eddfd98 (diff) | |
| download | bcm5719-llvm-d84a63398e7cb72816f3e2189892af705340bcd8.tar.gz bcm5719-llvm-d84a63398e7cb72816f3e2189892af705340bcd8.zip | |
LoopVectorize: Simplify code. No functionality change.
llvm-svn: 182100
Diffstat (limited to 'llvm/lib/Transforms')
| -rw-r--r-- | llvm/lib/Transforms/Vectorize/LoopVectorize.cpp | 26 | 
1 files changed, 5 insertions, 21 deletions
| diff --git a/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp b/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp index 88e76dabfb5..fce2cf211cd 100644 --- a/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp +++ b/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp @@ -354,24 +354,10 @@ bool LoadHoisting::isHoistableLoad(Instruction *L) {  static void addMemAccesses(BasicBlock *BB, SmallPtrSet<Value *, 8> &Set) {    for (BasicBlock::iterator BI = BB->begin(), BE = BB->end(); BI != BE; ++BI) { -    Instruction *I = &*BI; -    Value *Addr = 0; - -    // Try a load. -    LoadInst *LI = dyn_cast<LoadInst>(I); -    if (LI) { -      Addr = LI->getPointerOperand(); -      Set.insert(Addr); -      continue; -    } - -    // Try a store. -    StoreInst *SI = dyn_cast<StoreInst>(I); -    if (!SI) -      continue; - -    Addr = SI->getPointerOperand(); -    Set.insert(Addr); +    if (LoadInst *LI = dyn_cast<LoadInst>(BI)) // Try a load. +      Set.insert(LI->getPointerOperand()); +    else if (StoreInst *SI = dyn_cast<StoreInst>(BI)) // Try a store. +      Set.insert(SI->getPointerOperand());    }  } @@ -2708,9 +2694,7 @@ void LoopVectorizationLegality::collectLoopUniforms() {      Uniforms.insert(I);      // Insert all operands. -    for (int i = 0, Op = I->getNumOperands(); i < Op; ++i) { -      Worklist.push_back(I->getOperand(i)); -    } +    Worklist.insert(Worklist.end(), I->op_begin(), I->op_end());    }  } | 

