diff options
author | David Majnemer <david.majnemer@gmail.com> | 2015-01-13 10:14:57 +0000 |
---|---|---|
committer | David Majnemer <david.majnemer@gmail.com> | 2015-01-13 10:14:57 +0000 |
commit | cdf8a71d3698feb31b9ef86b16f8b80ceb0d1097 (patch) | |
tree | 59d61f30748091aa4c701ea3ac0732a70504e062 /clang/lib/Sema/SemaDecl.cpp | |
parent | c3691827c04df000d153854361ac3405e43850e8 (diff) | |
download | bcm5719-llvm-cdf8a71d3698feb31b9ef86b16f8b80ceb0d1097.tar.gz bcm5719-llvm-cdf8a71d3698feb31b9ef86b16f8b80ceb0d1097.zip |
Revert "Sema: An extern declaration can't be a redeclaration of a parameter"
This reverts commit r225780, we can't compile line 181 in
sanitizer_platform_limits_posix.cc with this commit.
llvm-svn: 225781
Diffstat (limited to 'clang/lib/Sema/SemaDecl.cpp')
-rw-r--r-- | clang/lib/Sema/SemaDecl.cpp | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/clang/lib/Sema/SemaDecl.cpp b/clang/lib/Sema/SemaDecl.cpp index 82ffd4da5f8..baa6822da71 100644 --- a/clang/lib/Sema/SemaDecl.cpp +++ b/clang/lib/Sema/SemaDecl.cpp @@ -3282,12 +3282,14 @@ void Sema::MergeVarDecl(VarDecl *New, LookupResult &Previous) { } // Check if extern is followed by non-extern and vice-versa. - if (New->hasGlobalStorage() && !Old->hasLinkage() && Old->hasLocalStorage()) { + if (New->hasExternalStorage() && + !Old->hasLinkage() && Old->isLocalVarDecl()) { Diag(New->getLocation(), diag::err_extern_non_extern) << New->getDeclName(); Diag(OldLocation, PrevDiag); return New->setInvalidDecl(); } - if (Old->hasGlobalStorage() && !New->hasLinkage() && New->hasLocalStorage()) { + if (Old->hasLinkage() && New->isLocalVarDecl() && + !New->hasExternalStorage()) { Diag(New->getLocation(), diag::err_non_extern_extern) << New->getDeclName(); Diag(OldLocation, PrevDiag); return New->setInvalidDecl(); |