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 | |
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')
-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()); } } |