diff options
| author | Sebastian Redl <sebastian.redl@getdesigned.at> | 2012-02-19 16:31:05 +0000 |
|---|---|---|
| committer | Sebastian Redl <sebastian.redl@getdesigned.at> | 2012-02-19 16:31:05 +0000 |
| commit | 73cfbebed4c3a7d49b7256667e7e357ddcee65e2 (patch) | |
| tree | 449e5714eba2fde6b8f7ac6949d5aa8079f0b2af /clang/lib/Sema | |
| parent | 8558da19306a824f54108d026176cc4bdd8ee90e (diff) | |
| download | bcm5719-llvm-73cfbebed4c3a7d49b7256667e7e357ddcee65e2.tar.gz bcm5719-llvm-73cfbebed4c3a7d49b7256667e7e357ddcee65e2.zip | |
Emit a warning when list-initializing a std::initializer_list member.
llvm-svn: 150933
Diffstat (limited to 'clang/lib/Sema')
| -rw-r--r-- | clang/lib/Sema/SemaDeclCXX.cpp | 9 | ||||
| -rw-r--r-- | clang/lib/Sema/SemaExprCXX.cpp | 2 |
2 files changed, 10 insertions, 1 deletions
diff --git a/clang/lib/Sema/SemaDeclCXX.cpp b/clang/lib/Sema/SemaDeclCXX.cpp index 0b785f30990..48dc911b6a3 100644 --- a/clang/lib/Sema/SemaDeclCXX.cpp +++ b/clang/lib/Sema/SemaDeclCXX.cpp @@ -1643,6 +1643,10 @@ Sema::ActOnCXXInClassMemberInitializer(Decl *D, SourceLocation EqualLoc, ExprResult Init = InitExpr; if (!FD->getType()->isDependentType() && !InitExpr->isTypeDependent()) { + if (isa<InitListExpr>(InitExpr) && isStdInitializerList(FD->getType(), 0)) { + Diag(FD->getLocation(), diag::warn_dangling_std_initializer_list) + << /*at end of ctor*/1 << InitExpr->getSourceRange(); + } // FIXME: if there is no EqualLoc, this is list-initialization. Init = PerformCopyInitialization( InitializedEntity::InitializeMember(FD), EqualLoc, InitExpr); @@ -2112,6 +2116,11 @@ Sema::BuildMemberInitializer(ValueDecl *Member, Expr *Init, InitList = true; Args = &Init; NumArgs = 1; + + if (isStdInitializerList(Member->getType(), 0)) { + Diag(IdLoc, diag::warn_dangling_std_initializer_list) + << /*at end of ctor*/1 << InitRange; + } } // Initialize the member. diff --git a/clang/lib/Sema/SemaExprCXX.cpp b/clang/lib/Sema/SemaExprCXX.cpp index 5ec96be4870..c33ba7558b8 100644 --- a/clang/lib/Sema/SemaExprCXX.cpp +++ b/clang/lib/Sema/SemaExprCXX.cpp @@ -1087,7 +1087,7 @@ Sema::BuildCXXNew(SourceLocation StartLoc, bool UseGlobal, if (initStyle == CXXNewExpr::ListInit && isStdInitializerList(AllocType, 0)) { Diag(AllocTypeInfo->getTypeLoc().getBeginLoc(), diag::warn_dangling_std_initializer_list) - << /*at end of FE*/0 << Inits[0]->getSourceRange(); + << /*at end of FE*/0 << Inits[0]->getSourceRange(); } // In ARC, infer 'retaining' for the allocated |
