From d7222ec801618a1e22285832e7101de3dbc126d0 Mon Sep 17 00:00:00 2001 From: Chris Lattner Date: Thu, 6 Nov 2003 19:18:49 +0000 Subject: Fix bug: PR93 git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@9752 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Transforms/IPO/LowerSetJmp.cpp | 29 +++++++++++++++++++++++++++++ 1 file changed, 29 insertions(+) (limited to 'lib/Transforms/IPO/LowerSetJmp.cpp') diff --git a/lib/Transforms/IPO/LowerSetJmp.cpp b/lib/Transforms/IPO/LowerSetJmp.cpp index 01ff046f11..abbc2c92c4 100644 --- a/lib/Transforms/IPO/LowerSetJmp.cpp +++ b/lib/Transforms/IPO/LowerSetJmp.cpp @@ -41,6 +41,7 @@ #include "llvm/Pass.h" #include "llvm/Support/CFG.h" #include "llvm/Support/InstVisitor.h" +#include "llvm/Transforms/Utils/DemoteRegToStack.h" #include "Support/DepthFirstIterator.h" #include "Support/Statistic.h" #include "Support/StringExtras.h" @@ -377,6 +378,34 @@ void LowerSetJmp::TransformSetJmpCall(CallInst* Inst) SetJmpIDMap[Func]++), 0), "", Inst); + // We are guaranteed that there are no values live across basic blocks + // (because we are "not in SSA form" yet), but there can still be values live + // in basic blocks. Because of this, splitting the setjmp block can cause + // values above the setjmp to not dominate uses which are after the setjmp + // call. For all of these occasions, we must spill the value to the stack. + // + std::set InstrsAfterCall; + + // The call is probably very close to the end of the basic block, for the + // common usage pattern of: 'if (setjmp(...))', so keep track of the + // instructions after the call. + for (BasicBlock::iterator I = ++BasicBlock::iterator(Inst), E = ABlock->end(); + I != E; ++I) + InstrsAfterCall.insert(I); + + for (BasicBlock::iterator II = ABlock->begin(); + II != BasicBlock::iterator(Inst); ++II) + // Loop over all of the uses of instruction. If any of them are after the + // call, "spill" the value to the stack. + for (Value::use_iterator UI = II->use_begin(), E = II->use_end(); + UI != E; ++UI) + if (cast(*UI)->getParent() != ABlock || + InstrsAfterCall.count(cast(*UI))) { + DemoteRegToStack(*II); + break; + } + InstrsAfterCall.clear(); + // Change the setjmp call into a branch statement. We'll remove the // setjmp call in a little bit. No worries. BasicBlock* SetJmpContBlock = ABlock->splitBasicBlock(Inst); -- cgit v1.2.3