diff options
author | Dale Johannesen <dalej@apple.com> | 2008-05-01 22:27:44 +0000 |
---|---|---|
committer | Dale Johannesen <dalej@apple.com> | 2008-05-01 22:27:44 +0000 |
commit | 796026711db8342aead8d85cdd50d0dc2da59c4c (patch) | |
tree | feaef2b57701f594a497f835aa5dd210502e6f54 /lib/Transforms/Scalar/LoopRotation.cpp | |
parent | e85b7585e92b825eb549196a6567ad5c1678518b (diff) | |
download | llvm-796026711db8342aead8d85cdd50d0dc2da59c4c.tar.gz llvm-796026711db8342aead8d85cdd50d0dc2da59c4c.tar.bz2 llvm-796026711db8342aead8d85cdd50d0dc2da59c4c.tar.xz |
Don't try to create PHIs of struct types. Fallout
from x86-64 calling convention work.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@50545 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms/Scalar/LoopRotation.cpp')
-rw-r--r-- | lib/Transforms/Scalar/LoopRotation.cpp | 39 |
1 files changed, 31 insertions, 8 deletions
diff --git a/lib/Transforms/Scalar/LoopRotation.cpp b/lib/Transforms/Scalar/LoopRotation.cpp index 51e2cd8900..0e31ece4b3 100644 --- a/lib/Transforms/Scalar/LoopRotation.cpp +++ b/lib/Transforms/Scalar/LoopRotation.cpp @@ -249,14 +249,37 @@ bool LoopRotate::rotateLoop(Loop *Lp, LPPassManager &LPM) { // create new PHINode for this instruction. Instruction *NewHeaderReplacement = NULL; if (usedOutsideOriginalHeader(In)) { - PHINode *PN = PHINode::Create(In->getType(), In->getName()); - PN->addIncoming(In, OrigHeader); - PN->addIncoming(C, OrigPreHeader); - NewHeader->getInstList().push_front(PN); - NewHeaderReplacement = PN; - } - - // "In" can be replaced by NPH or NH at various places. + const StructType *STy = dyn_cast<StructType>(In->getType()); + if (STy) { + // Can't create PHI nodes for this type. If there are any getResults + // not defined in this block, move them back to this block. PHI + // nodes will be created for all getResults later. + BasicBlock::iterator InsertPoint; + if (InvokeInst *II = dyn_cast<InvokeInst>(In)) { + InsertPoint = II->getNormalDest()->begin(); + while (isa<PHINode>(InsertPoint)) + ++InsertPoint; + } else { + InsertPoint = I; // call + InsertPoint++; + } + for (Value::use_iterator UI = In->use_begin(), UE = In->use_end(); + UI != UE; ++UI) { + GetResultInst *InGR = cast<GetResultInst>(UI); + if (InGR->getParent() != OrigHeader) { + // move InGR to immediately follow call. It will be picked + // up, cloned and PHI'd on the next iteration. + InGR->moveBefore(InsertPoint); + } + } + } else { + PHINode *PN = PHINode::Create(In->getType(), In->getName()); + PN->addIncoming(In, OrigHeader); + PN->addIncoming(C, OrigPreHeader); + NewHeader->getInstList().push_front(PN); + NewHeaderReplacement = PN; + } + } LoopHeaderInfo.push_back(RenameData(In, C, NewHeaderReplacement)); } |