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/CGDeclCXX.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/CGDeclCXX.cpp')
-rw-r--r-- | lib/CodeGen/CGDeclCXX.cpp | 34 |
1 files changed, 8 insertions, 26 deletions
diff --git a/lib/CodeGen/CGDeclCXX.cpp b/lib/CodeGen/CGDeclCXX.cpp index cbee247238..036be92885 100644 --- a/lib/CodeGen/CGDeclCXX.cpp +++ b/lib/CodeGen/CGDeclCXX.cpp @@ -321,11 +321,7 @@ CodeGenFunction::EmitStaticCXXBlockVarDeclInit(const VarDecl &D, EmitBlock(InitCheckBlock); - // Variables used when coping with thread-safe statics and exceptions. - llvm::BasicBlock *SavedLandingPad = 0; - llvm::BasicBlock *Abort = 0; - - if (ThreadsafeStatics) { + if (ThreadsafeStatics) { // Call __cxa_guard_acquire. V = Builder.CreateCall(getGuardAcquireFn(*this), GuardVariable); @@ -334,13 +330,14 @@ CodeGenFunction::EmitStaticCXXBlockVarDeclInit(const VarDecl &D, Builder.CreateCondBr(Builder.CreateIsNotNull(V, "tobool"), InitBlock, EndBlock); + EmitBlock(InitBlock); + if (Exceptions) { - SavedLandingPad = getInvokeDest(); - Abort = createBasicBlock("guard.abort"); - setInvokeDest(Abort); - } + EHCleanupBlock Cleanup(*this); - EmitBlock(InitBlock); + // Call __cxa_guard_abort. + Builder.CreateCall(getGuardAbortFn(*this), GuardVariable); + } } if (D.getType()->isReferenceType()) { @@ -356,7 +353,7 @@ CodeGenFunction::EmitStaticCXXBlockVarDeclInit(const VarDecl &D, if (ThreadsafeStatics) { // Call __cxa_guard_release. - Builder.CreateCall(getGuardReleaseFn(*this), GuardVariable); + Builder.CreateCall(getGuardReleaseFn(*this), GuardVariable); } else { llvm::Value *One = llvm::ConstantInt::get(llvm::Type::getInt8Ty(VMContext), 1); @@ -367,20 +364,5 @@ CodeGenFunction::EmitStaticCXXBlockVarDeclInit(const VarDecl &D, if (!D.getType()->isReferenceType()) EmitDeclDestroy(*this, D, GV); - if (ThreadsafeStatics && Exceptions) { - // If an exception is thrown during initialization, call __cxa_guard_abort - // along the exceptional edge. - EmitBranch(EndBlock); - - EmitBlock(Abort); - - // Call __cxa_guard_abort along the exceptional edge. - Builder.CreateCall(getGuardAbortFn(*this), GuardVariable); - setInvokeDest(SavedLandingPad); - - // Rethrow the current exception. - EmitRethrow(); - } - EmitBlock(EndBlock); } |