diff options
| author | Richard Smith <richard-llvm@metafoo.co.uk> | 2012-01-06 02:30:50 +0000 |
|---|---|---|
| committer | Richard Smith <richard-llvm@metafoo.co.uk> | 2012-01-06 02:30:50 +0000 |
| commit | b3851f5ca15e7b3d0c8a272b1bd4d58bf99fedac (patch) | |
| tree | 6be913a799a0e8e2f6c3c5d21711f44ea8cbf9b1 /clang/lib/Sema | |
| parent | 06c9397665089514e4738e31f58aa2341c1a5527 (diff) | |
| download | bcm5719-llvm-b3851f5ca15e7b3d0c8a272b1bd4d58bf99fedac.tar.gz bcm5719-llvm-b3851f5ca15e7b3d0c8a272b1bd4d58bf99fedac.zip | |
David Blaikie and Chandler would like us to diagnose
int f();
in function scopes under -Wvexing-parse, so now we do.
llvm-svn: 147649
Diffstat (limited to 'clang/lib/Sema')
| -rw-r--r-- | clang/lib/Sema/SemaDecl.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/clang/lib/Sema/SemaDecl.cpp b/clang/lib/Sema/SemaDecl.cpp index 3e9e66d55dc..fecfce4d2d2 100644 --- a/clang/lib/Sema/SemaDecl.cpp +++ b/clang/lib/Sema/SemaDecl.cpp @@ -4914,8 +4914,7 @@ Sema::ActOnFunctionDeclarator(Scope *S, Declarator &D, DeclContext *DC, == DeclSpec::SCS_unspecified) { QualType T = R->getAs<FunctionType>()->getResultType(); DeclaratorChunk &C = D.getTypeObject(0); - if ((T->isDependentType() || T->isRecordType()) && - C.Fun.NumArgs == 0 && !C.Fun.isVariadic && + if (!T->isVoidType() && C.Fun.NumArgs == 0 && !C.Fun.isVariadic && !C.Fun.TrailingReturnType && C.Fun.getExceptionSpecType() == EST_None) { Diag(C.Loc, diag::warn_empty_parens_are_function_decl) |
