maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

[LoopUnroll] Use addClonedBlockToLoopInfo to add loop header to LI (NFC). 

Summary:
I have a similar patch up for review already (D29173). If you prefer I
can squash them both together.

Also I think there more potential for code sharing between
LoopUnroll.cpp and LoopUnrollRuntime.cpp. Do you think patches for
that would be worthwhile? 

Reviewers: mkuper, mzolotukhin

Reviewed By: mkuper, mzolotukhin

Subscribers: llvm-commits

Differential Revision: https://reviews.llvm.org/D29311

llvm-svn: 293758
This commit is contained in:
Florian Hahn 2017-02-01 10:39:35 +00:00
parent 3a52eb0054
commit a35b8a4852
1 changed files with 8 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
llvm/lib/Transforms/Utils/LoopUnroll.cpp
View File

@ -472,19 +472,16 @@ bool llvm::UnrollLoop(Loop *L, unsigned Count, unsigned TripCount, bool Force,
BasicBlock *New = CloneBasicBlock(*BB, VMap, "." + Twine(It)); BasicBlock *New = CloneBasicBlock(*BB, VMap, "." + Twine(It));
Header->getParent()->getBasicBlockList().push_back(New); Header->getParent()->getBasicBlockList().push_back(New);
assert(*BB != Header || LI->getLoopFor(*BB) == L &&
"Header should not be in a sub-loop");
// Tell LI about New. // Tell LI about New.
if (*BB == Header) { const Loop *OldLoop = addClonedBlockToLoopInfo(*BB, New, LI, NewLoops);
assert(LI->getLoopFor(*BB) == L && "Header should not be in a sub-loop"); if (OldLoop) {
L->addBasicBlockToLoop(New, *LI); LoopsToSimplify.insert(NewLoops[OldLoop]);
} else {
const Loop *OldLoop = addClonedBlockToLoopInfo(*BB, New, LI, NewLoops);
if (OldLoop) {
LoopsToSimplify.insert(NewLoops[OldLoop]);
// Forget the old loop, since its inputs may have changed. // Forget the old loop, since its inputs may have changed.
if (SE) if (SE)
SE->forgetLoop(OldLoop); SE->forgetLoop(OldLoop);
}
} }
if (*BB == Header) if (*BB == Header)