diff options
| author | Michael Zolotukhin <mzolotukhin@apple.com> | 2015-07-08 21:47:47 +0000 |
|---|---|---|
| committer | Michael Zolotukhin <mzolotukhin@apple.com> | 2015-07-08 21:47:47 +0000 |
| commit | 7db3063f8799b64441ab4d30849c9f34eb951ab8 (patch) | |
| tree | d2e41e18a7cbc604813d4c4f63921ebd766fe259 /llvm/lib/Transforms | |
| parent | df08179d1bf128bd0d16dfa68b37cb409930eb3c (diff) | |
| download | bcm5719-llvm-7db3063f8799b64441ab4d30849c9f34eb951ab8.tar.gz bcm5719-llvm-7db3063f8799b64441ab4d30849c9f34eb951ab8.zip | |
[LoopVectorizer] Remove redundant variable LastBypassBlock. NFC.
llvm-svn: 241738
Diffstat (limited to 'llvm/lib/Transforms')
| -rw-r--r-- | llvm/lib/Transforms/Vectorize/LoopVectorize.cpp | 26 |
1 files changed, 12 insertions, 14 deletions
diff --git a/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp b/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp index 5ba14174ac7..f1f18613d9e 100644 --- a/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp +++ b/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp @@ -2647,22 +2647,20 @@ void InnerLoopVectorizer::createEmptyLoop() { Value *Cmp = BypassBuilder.CreateICmpEQ(IdxEndRoundDown, StartIdx, "cmp.zero"); - BasicBlock *LastBypassBlock = BypassBlock; - // Generate code to check that the loops trip count that we computed by adding // one to the backedge-taken count will not overflow. { auto PastOverflowCheck = std::next(BasicBlock::iterator(OverflowCheckAnchor)); BasicBlock *CheckBlock = - LastBypassBlock->splitBasicBlock(PastOverflowCheck, "overflow.checked"); + BypassBlock->splitBasicBlock(PastOverflowCheck, "overflow.checked"); if (ParentLoop) ParentLoop->addBasicBlockToLoop(CheckBlock, *LI); LoopBypassBlocks.push_back(CheckBlock); ReplaceInstWithInst( - LastBypassBlock->getTerminator(), + BypassBlock->getTerminator(), BranchInst::Create(ScalarPH, CheckBlock, CheckBCOverflow)); - LastBypassBlock = CheckBlock; + BypassBlock = CheckBlock; } // Generate the code to check that the strides we assumed to be one are really @@ -2671,23 +2669,23 @@ void InnerLoopVectorizer::createEmptyLoop() { Instruction *StrideCheck; Instruction *FirstCheckInst; std::tie(FirstCheckInst, StrideCheck) = - addStrideCheck(LastBypassBlock->getTerminator()); + addStrideCheck(BypassBlock->getTerminator()); if (StrideCheck) { AddedSafetyChecks = true; // Create a new block containing the stride check. BasicBlock *CheckBlock = - LastBypassBlock->splitBasicBlock(FirstCheckInst, "vector.stridecheck"); + BypassBlock->splitBasicBlock(FirstCheckInst, "vector.stridecheck"); if (ParentLoop) ParentLoop->addBasicBlockToLoop(CheckBlock, *LI); LoopBypassBlocks.push_back(CheckBlock); // Replace the branch into the memory check block with a conditional branch // for the "few elements case". - ReplaceInstWithInst(LastBypassBlock->getTerminator(), + ReplaceInstWithInst(BypassBlock->getTerminator(), BranchInst::Create(MiddleBlock, CheckBlock, Cmp)); Cmp = StrideCheck; - LastBypassBlock = CheckBlock; + BypassBlock = CheckBlock; } // Generate the code that checks in runtime if arrays overlap. We put the @@ -2695,26 +2693,26 @@ void InnerLoopVectorizer::createEmptyLoop() { // faster. Instruction *MemRuntimeCheck; std::tie(FirstCheckInst, MemRuntimeCheck) = - Legal->getLAI()->addRuntimeCheck(LastBypassBlock->getTerminator()); + Legal->getLAI()->addRuntimeCheck(BypassBlock->getTerminator()); if (MemRuntimeCheck) { AddedSafetyChecks = true; // Create a new block containing the memory check. BasicBlock *CheckBlock = - LastBypassBlock->splitBasicBlock(FirstCheckInst, "vector.memcheck"); + BypassBlock->splitBasicBlock(FirstCheckInst, "vector.memcheck"); if (ParentLoop) ParentLoop->addBasicBlockToLoop(CheckBlock, *LI); LoopBypassBlocks.push_back(CheckBlock); // Replace the branch into the memory check block with a conditional branch // for the "few elements case". - ReplaceInstWithInst(LastBypassBlock->getTerminator(), + ReplaceInstWithInst(BypassBlock->getTerminator(), BranchInst::Create(MiddleBlock, CheckBlock, Cmp)); Cmp = MemRuntimeCheck; - LastBypassBlock = CheckBlock; + BypassBlock = CheckBlock; } - ReplaceInstWithInst(LastBypassBlock->getTerminator(), + ReplaceInstWithInst(BypassBlock->getTerminator(), BranchInst::Create(MiddleBlock, VectorPH, Cmp)); // We are going to resume the execution of the scalar loop. |

