diff options
author | Anders Carlsson <andersca@mac.com> | 2010-02-06 19:50:17 +0000 |
---|---|---|
committer | Anders Carlsson <andersca@mac.com> | 2010-02-06 19:50:17 +0000 |
commit | ba63167b64656e5b019987ded8cd639bbb4c0867 (patch) | |
tree | 8ea0285a349a86419635d2ad1edba440c5d7a86a /clang | |
parent | d6fff557db24f2d08340564b14872c6d70cd9650 (diff) | |
download | bcm5719-llvm-ba63167b64656e5b019987ded8cd639bbb4c0867.tar.gz bcm5719-llvm-ba63167b64656e5b019987ded8cd639bbb4c0867.zip |
If a constructor throws an exception we need to execute the destructors for all fully constructed members. Fixes ctor_dtor_count.cpp in the test suite.
llvm-svn: 95501
Diffstat (limited to 'clang')
-rw-r--r-- | clang/lib/CodeGen/CGClass.cpp | 19 |
1 files changed, 19 insertions, 0 deletions
diff --git a/clang/lib/CodeGen/CGClass.cpp b/clang/lib/CodeGen/CGClass.cpp index 3804a7cf830..4ec524da8b5 100644 --- a/clang/lib/CodeGen/CGClass.cpp +++ b/clang/lib/CodeGen/CGClass.cpp @@ -849,6 +849,25 @@ static void EmitMemberInitializer(CodeGenFunction &CGF, } else { CGF.EmitAggExpr(MemberInit->getInit(), LHS.getAddress(), LHS.isVolatileQualified(), false, true); + + if (!CGF.Exceptions) + return; + + const RecordType *RT = FieldType->getAs<RecordType>(); + if (!RT) + return; + + CXXRecordDecl *RD = cast<CXXRecordDecl>(RT->getDecl()); + if (!RD->hasTrivialDestructor()) { + // FIXME: Is this OK for C++0x delegating constructors? + CodeGenFunction::EHCleanupBlock Cleanup(CGF); + + llvm::Value *ThisPtr = CGF.LoadCXXThis(); + LValue LHS = CGF.EmitLValueForField(ThisPtr, Field, 0); + + CXXDestructorDecl *DD = RD->getDestructor(CGF.getContext()); + CGF.EmitCXXDestructorCall(DD, Dtor_Complete, LHS.getAddress()); + } } } |