From: Daniel Dunbar <daniel@zuster.org>
Date: Fri, 13 Feb 2009 22:48:56 +0000 (+0000)
Subject: Warn about attribute used ignored on "extern int a
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=186204bfcf9c53d48143ec300d4c3d036fed4140;p=clang

Warn about attribute used ignored on "extern int a
__attribute__((used))".


git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@64499 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/Sema/SemaDeclAttr.cpp b/lib/Sema/SemaDeclAttr.cpp
index 29f6cb18a0..3ea2b5064c 100644
--- a/lib/Sema/SemaDeclAttr.cpp
+++ b/lib/Sema/SemaDeclAttr.cpp
@@ -439,7 +439,7 @@ static void HandleUsedAttr(Decl *d, const AttributeList &Attr, Sema &S) {
   }
   
   if (const VarDecl *VD = dyn_cast<VarDecl>(d)) {
-    if (VD->hasLocalStorage()) {
+    if (VD->hasLocalStorage() || VD->hasExternalStorage()) {
       S.Diag(Attr.getLoc(), diag::warn_attribute_ignored) << "used";
       return;
     }
diff --git a/test/Sema/attr-used.c b/test/Sema/attr-used.c
index 7d0265ef32..10674814ce 100644
--- a/test/Sema/attr-used.c
+++ b/test/Sema/attr-used.c
@@ -1,5 +1,8 @@
 // RUN: clang -verify -fsyntax-only %s
 
+extern int l0 __attribute__((used)); // expected-warning {{used attribute ignored}}
+__private_extern__ int l1 __attribute__((used)); // expected-warning {{used attribute ignored}}
+
 struct __attribute__((used)) s { // expected-warning {{'used' attribute only applies to variable and function types}}
   int x;
 };