diff options
author | David Majnemer <david.majnemer@gmail.com> | 2015-01-14 00:31:13 +0000 |
---|---|---|
committer | David Majnemer <david.majnemer@gmail.com> | 2015-01-14 00:31:13 +0000 |
commit | 3806cf84dce82865ff786f837280441bdb59ead0 (patch) | |
tree | 114d9bc6afd9bc0aa5f84f84109c9b74f75cc9dd | |
parent | 5fe8dcb8d847c5a512fd3861942dac38317aa4d4 (diff) |
Sema: An extern declaration can't be a redeclaration of a parameter
In the following:
void f(int x) { extern int x; }
The second declaration of 'x' shouldn't be considered a redeclaration of
the parameter.
This is a different approach to r225780.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@225875 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r-- | include/clang/AST/Decl.h | 5 | ||||
-rw-r--r-- | lib/Sema/SemaDecl.cpp | 4 | ||||
-rw-r--r-- | test/Sema/var-redecl.c | 4 |
3 files changed, 11 insertions, 2 deletions
diff --git a/include/clang/AST/Decl.h b/include/clang/AST/Decl.h index cdba3bda80..a39888f9e1 100644 --- a/include/clang/AST/Decl.h +++ b/include/clang/AST/Decl.h @@ -906,6 +906,11 @@ public: return false; } + /// \brief Similar to isLocalVarDecl but also includes parameters. + bool isLocalVarDeclOrParm() const { + return isLocalVarDecl() || getKind() == Decl::ParmVar; + } + /// isFunctionOrMethodVarDecl - Similar to isLocalVarDecl, but /// excludes variables declared in blocks. bool isFunctionOrMethodVarDecl() const { diff --git a/lib/Sema/SemaDecl.cpp b/lib/Sema/SemaDecl.cpp index 7d1e28ee59..1bf57f6cee 100644 --- a/lib/Sema/SemaDecl.cpp +++ b/lib/Sema/SemaDecl.cpp @@ -3285,12 +3285,12 @@ void Sema::MergeVarDecl(VarDecl *New, LookupResult &Previous) { // Check if extern is followed by non-extern and vice-versa. if (New->hasExternalStorage() && - !Old->hasLinkage() && Old->isLocalVarDecl()) { + !Old->hasLinkage() && Old->isLocalVarDeclOrParm()) { Diag(New->getLocation(), diag::err_extern_non_extern) << New->getDeclName(); Diag(OldLocation, PrevDiag); return New->setInvalidDecl(); } - if (Old->hasLinkage() && New->isLocalVarDecl() && + if (Old->hasLinkage() && New->isLocalVarDeclOrParm() && !New->hasExternalStorage()) { Diag(New->getLocation(), diag::err_non_extern_extern) << New->getDeclName(); Diag(OldLocation, PrevDiag); diff --git a/test/Sema/var-redecl.c b/test/Sema/var-redecl.c index 0e30aa2713..5ba6965f71 100644 --- a/test/Sema/var-redecl.c +++ b/test/Sema/var-redecl.c @@ -60,3 +60,7 @@ int *p=&g19; // expected-error{{use of undeclared identifier 'g19'}} \ static int a; extern int a; // expected-note {{previous declaration is here}} int a; // expected-error {{non-static declaration of 'a' follows static declaration}} + +void f(int x) { // expected-note {{previous definition is here}} + extern int x; // expected-error {{extern declaration of 'x' follows non-extern declaration}} +} |