diff options
| author | Eli Friedman <eli.friedman@gmail.com> | 2009-05-16 11:45:48 +0000 |
|---|---|---|
| committer | Eli Friedman <eli.friedman@gmail.com> | 2009-05-16 11:45:48 +0000 |
| commit | 0b4af8f7554d8ae782fbd9e92a6d4211e8c95c2a (patch) | |
| tree | af7500e64aa19393c7e7475db881534a99a3d138 /clang/lib/Sema | |
| parent | 18bb9284fffb62977647bffaf0bd5fec17241751 (diff) | |
| download | bcm5719-llvm-0b4af8f7554d8ae782fbd9e92a6d4211e8c95c2a.tar.gz bcm5719-llvm-0b4af8f7554d8ae782fbd9e92a6d4211e8c95c2a.zip | |
PR3009: Get rid of bogus warning for scalar compound literals.
This patch isn't quite ideal in that it eliminates the warning for
constructs like "int a = {1};", where the braces are in fact redundant.
However, that would have required a bunch of refactoring, and it's
much less likely to cause confusion compared to redundant nested braces.
llvm-svn: 71939
Diffstat (limited to 'clang/lib/Sema')
| -rw-r--r-- | clang/lib/Sema/SemaInit.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/clang/lib/Sema/SemaInit.cpp b/clang/lib/Sema/SemaInit.cpp index a8a0ea3aeac..17113f68bd4 100644 --- a/clang/lib/Sema/SemaInit.cpp +++ b/clang/lib/Sema/SemaInit.cpp @@ -538,7 +538,7 @@ void InitListChecker::CheckExplicitInitList(InitListExpr *IList, QualType &T, } } - if (T->isScalarType()) + if (T->isScalarType() && !TopLevelObject) SemaRef.Diag(IList->getLocStart(), diag::warn_braces_around_scalar_init) << IList->getSourceRange() << CodeModificationHint::CreateRemoval(SourceRange(IList->getLocStart())) |
