diff options
author | Douglas Gregor <dgregor@apple.com> | 2010-05-16 00:44:00 +0000 |
---|---|---|
committer | Douglas Gregor <dgregor@apple.com> | 2010-05-16 00:44:00 +0000 |
commit | 1eb2e59338c4b9c0429fc39ca98662adc9e7a3f2 (patch) | |
tree | 126954184e0133b6da5ad917cc4d5b10a52c7657 /lib/CodeGen/CGException.cpp | |
parent | c9a8fa4e4df099cf0addc55d9ba513926588a0fb (diff) |
Revert r103880 (thread-safe static initialization w/ exceptions),
because it's causing strange linker errors. Unfixes PR7144.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@103890 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/CodeGen/CGException.cpp')
-rw-r--r-- | lib/CodeGen/CGException.cpp | 27 |
1 files changed, 11 insertions, 16 deletions
diff --git a/lib/CodeGen/CGException.cpp b/lib/CodeGen/CGException.cpp index 76a57d6f55..c1d05bf233 100644 --- a/lib/CodeGen/CGException.cpp +++ b/lib/CodeGen/CGException.cpp @@ -239,23 +239,19 @@ static void CopyObject(CodeGenFunction &CGF, QualType ObjectType, } } -void CodeGenFunction::EmitRethrow() { - if (getInvokeDest()) { - llvm::BasicBlock *Cont = createBasicBlock("invoke.cont"); - Builder.CreateInvoke(getReThrowFn(*this), Cont, getInvokeDest()) - ->setDoesNotReturn(); - EmitBlock(Cont); - } else - Builder.CreateCall(getReThrowFn(*this))->setDoesNotReturn(); - Builder.CreateUnreachable(); - - // Clear the insertion point to indicate we are in unreachable code. - Builder.ClearInsertionPoint(); -} - void CodeGenFunction::EmitCXXThrowExpr(const CXXThrowExpr *E) { if (!E->getSubExpr()) { - EmitRethrow(); + if (getInvokeDest()) { + llvm::BasicBlock *Cont = createBasicBlock("invoke.cont"); + Builder.CreateInvoke(getReThrowFn(*this), Cont, getInvokeDest()) + ->setDoesNotReturn(); + EmitBlock(Cont); + } else + Builder.CreateCall(getReThrowFn(*this))->setDoesNotReturn(); + Builder.CreateUnreachable(); + + // Clear the insertion point to indicate we are in unreachable code. + Builder.ClearInsertionPoint(); return; } @@ -566,7 +562,6 @@ void CodeGenFunction::ExitCXXTryStmt(const CXXTryStmt &S, // FIXME: we need to do this sooner so that the EH region for the // cleanup doesn't start until after the ctor completes, use a decl // init? - // FIXME: Alternatively, can we just elide this copy entirely? CopyObject(*this, CatchParam->getType().getNonReferenceType(), WasPointer, WasPointerReference, ExcObject, GetAddrOfLocalVar(CatchParam)); |