From: David Majnemer <david.majnemer@gmail.com>
Date: Tue, 13 Jan 2015 10:14:57 +0000 (+0000)
Subject: Revert "Sema: An extern declaration can't be a redeclaration of a parameter"
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=cdf8a71d3698feb31b9ef86b16f8b80ceb0d1097;p=platform%2Fupstream%2Fllvm.git

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
---

diff --git a/clang/lib/Sema/SemaDecl.cpp b/clang/lib/Sema/SemaDecl.cpp
index 82ffd4d..baa6822 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();
diff --git a/clang/test/Sema/private-extern.c b/clang/test/Sema/private-extern.c
index 94afd87..0c13c92 100644
--- a/clang/test/Sema/private-extern.c
+++ b/clang/test/Sema/private-extern.c
@@ -83,7 +83,3 @@ __private_extern__ int g19;
 int g19 = 0;
 
 __private_extern__ int g20 = 0;
-
-void f10(int g20) { // expected-note{{previous definition is here}}
-  extern int g20; // expected-error{{extern declaration of 'g20' follows non-extern declaration}}
-}