diff --git a/lib/CodeGen/CGClass.cpp b/lib/CodeGen/CGClass.cpp index f269b675a0d656dfbb7712a01cd2f737cb63fa77..2cea434f62de1d891f2450358bd9318b2d460a7a 100644 --- a/lib/CodeGen/CGClass.cpp +++ b/lib/CodeGen/CGClass.cpp @@ -456,7 +456,7 @@ static void EmitAggMemberInitializer(CodeGenFunction &CGF, // Now, outside of the initializer cleanup scope, destroy the backing array // for a std::initializer_list member. - CGF.MaybeEmitStdInitializerListCleanup(LV, Init); + CGF.MaybeEmitStdInitializerListCleanup(LV.getAddress(), Init); return; } diff --git a/lib/CodeGen/CGDecl.cpp b/lib/CodeGen/CGDecl.cpp index 83d74106f3b6ab089f7a2b3b945ddd0ba1b2cd2d..fcea927c63ada05eacaf9f9d60ba51d8f2b46f6a 100644 --- a/lib/CodeGen/CGDecl.cpp +++ b/lib/CodeGen/CGDecl.cpp @@ -1093,7 +1093,7 @@ void CodeGenFunction::EmitExprAsInit(const Expr *init, AggValueSlot::IsDestructed, AggValueSlot::DoesNotNeedGCBarriers, AggValueSlot::IsNotAliased)); - MaybeEmitStdInitializerListCleanup(lvalue, init); + MaybeEmitStdInitializerListCleanup(lvalue.getAddress(), init); } } diff --git a/lib/CodeGen/CGExprAgg.cpp b/lib/CodeGen/CGExprAgg.cpp index 13f5bc47d5b523082b57cc25a39a7778c13bc3cf..afe70a5dda8b353da5e12506be9eb654165d223c 100644 --- a/lib/CodeGen/CGExprAgg.cpp +++ b/lib/CodeGen/CGExprAgg.cpp @@ -1263,19 +1263,17 @@ void CodeGenFunction::EmitAggregateCopy(llvm::Value *DestPtr, Alignment, isVolatile); } -void CodeGenFunction::MaybeEmitStdInitializerListCleanup(LValue lvalue, - const Expr *init) { +void CodeGenFunction::MaybeEmitStdInitializerListCleanup(llvm::Value *loc, + const Expr *init) { const ExprWithCleanups *cleanups = dyn_cast<ExprWithCleanups>(init); - if (!cleanups) - return; // Nothing interesting here. - init = cleanups->getSubExpr(); + if (cleanups) + init = cleanups->getSubExpr(); if (isa<InitListExpr>(init) && cast<InitListExpr>(init)->initializesStdInitializerList()) { // We initialized this std::initializer_list with an initializer list. // A backing array was created. Push a cleanup for it. - EmitStdInitializerListCleanup(lvalue.getAddress(), - cast<InitListExpr>(init)); + EmitStdInitializerListCleanup(loc, cast<InitListExpr>(init)); } } diff --git a/lib/CodeGen/CGExprCXX.cpp b/lib/CodeGen/CGExprCXX.cpp index 548bbc9e617e8bee6b88e76ef8bffd84907ae7af..7c7bdd9eda48c5cd8c77f7a0ba74bf8ece2e0599 100644 --- a/lib/CodeGen/CGExprCXX.cpp +++ b/lib/CodeGen/CGExprCXX.cpp @@ -762,6 +762,8 @@ static void StoreAnyExprIntoOneUnit(CodeGenFunction &CGF, const CXXNewExpr *E, AggValueSlot::DoesNotNeedGCBarriers, AggValueSlot::IsNotAliased); CGF.EmitAggExpr(Init, Slot); + + CGF.MaybeEmitStdInitializerListCleanup(NewPtr, Init); } } diff --git a/lib/CodeGen/CodeGenFunction.h b/lib/CodeGen/CodeGenFunction.h index 95420e5344ed950b3fc4d55c7f4c2a59dec3256e..6327fa2225a9047ecda69209511ef92b28c419d8 100644 --- a/lib/CodeGen/CodeGenFunction.h +++ b/lib/CodeGen/CodeGenFunction.h @@ -1811,7 +1811,7 @@ public: llvm::Value* EmitCXXTypeidExpr(const CXXTypeidExpr *E); llvm::Value *EmitDynamicCast(llvm::Value *V, const CXXDynamicCastExpr *DCE); - void MaybeEmitStdInitializerListCleanup(LValue lvalue, const Expr *init); + void MaybeEmitStdInitializerListCleanup(llvm::Value *loc, const Expr *init); void EmitStdInitializerListCleanup(llvm::Value *loc, const InitListExpr *init); diff --git a/test/CodeGenCXX/cxx0x-initializer-stdinitializerlist.cpp b/test/CodeGenCXX/cxx0x-initializer-stdinitializerlist.cpp index 55e399223850b4d24c202c34d886919ebb82a6f9..2d7ff46ef79d0ce7a19f5a59817fcbde0c0b090d 100644 --- a/test/CodeGenCXX/cxx0x-initializer-stdinitializerlist.cpp +++ b/test/CodeGenCXX/cxx0x-initializer-stdinitializerlist.cpp @@ -195,3 +195,23 @@ haslist2::haslist2() // CHECK: call void @_ZN10destroyme2D1Ev // CHECK: call void @_ZN10destroyme1D1Ev } + +void fn10() { + // CHECK: define void @_Z4fn10v + // CHECK: alloca [3 x i32] + // CHECK: call noalias i8* @_Znwm + // CHECK: store i32 1 + // CHECK: store i32 2 + // CHECK: store i32 3 + // CHECK: store i32* + // CHECK: store i{{32|64}} 3 + (void) new std::initializer_list<int> {1, 2, 3}; +} + +void fn11() { + // CHECK: define void @_Z4fn11v + (void) new std::initializer_list<destroyme1> {destroyme1(), destroyme1()}; + // CHECK: call void @_ZN10destroyme1D1Ev + destroyme2 dm2; + // CHECK: call void @_ZN10destroyme2D1Ev +}