From: Richard Henderson <rth@redhat.com>
Date: Thu, 16 May 2002 00:14:24 +0000 (-0700)
Subject: varasm.c (merge_weak): Remove special case for extern and common.
X-Git-Tag: releases/gcc-3.3.0~5038
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=34fb9ba554ab82c877dc0475d78deb166c745941;p=thirdparty%2Fgcc.git

varasm.c (merge_weak): Remove special case for extern and common.

        * varasm.c (merge_weak): Remove special case for extern and common.

        * gcc.dg/weak-5.c (vfoo1c, vfoo1g): Warn here.

From-SVN: r53498
---

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index a83b9034d4e7..714b5fda51e4 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,7 @@
+2002-05-15  Richard Henderson  <rth@redhat.com>
+
+	* varasm.c (merge_weak): Remove special case for extern and common.
+
 2002-05-15  Matt Hiller  <hiller@redhat.com>
 
 	* testsuite/gcc.c-torture/compile/20000804-1.x: Don't return 1 if
diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 5676c95265e3..e2b5ff06f469 100644
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,7 @@
+2002-05-15  Richard Henderson  <rth@redhat.com>
+
+	* gcc.dg/weak-5.c (vfoo1c, vfoo1g): Warn here.
+
 2002-05-15  Richard Henderson  <rth@redhat.com>
 
 	* gcc.dg/weak-5.c (vfoo1c): No warning here.
diff --git a/gcc/testsuite/gcc.dg/weak-5.c b/gcc/testsuite/gcc.dg/weak-5.c
index e531b27beebc..8576379d3fb7 100644
--- a/gcc/testsuite/gcc.dg/weak-5.c
+++ b/gcc/testsuite/gcc.dg/weak-5.c
@@ -44,7 +44,7 @@ void * foo1c (void)
 {
   return (void *)&vfoo1c;
 }
-extern int vfoo1c __attribute__((weak));
+extern int vfoo1c __attribute__((weak)); /* { dg-warning "unspecified behavior" } */
 
 
 extern int vfoo1d __attribute__((weak));
@@ -68,7 +68,7 @@ void * foo1f (void)
 {
   return (void *)&vfoo1f;
 }
-extern int vfoo1f __attribute__((weak)); /* { dg-warning "weak declaration" "weak declaration" } */
+extern int vfoo1f __attribute__((weak)); /* { dg-warning "unspecified behavior" } */
 
 
 extern int vfoo1g;
@@ -76,7 +76,7 @@ void * foo1g (void)
 {
   return (void *)&vfoo1g;
 }
-int vfoo1g __attribute__((weak));
+int vfoo1g __attribute__((weak)); /* { dg-warning "unspecified behavior" } */
 
 
 extern int vfoo1h __attribute__((weak));
diff --git a/gcc/varasm.c b/gcc/varasm.c
index 05a8ada095b3..4a8841230282 100644
--- a/gcc/varasm.c
+++ b/gcc/varasm.c
@@ -4848,16 +4848,7 @@ merge_weak (newdecl, olddecl)
 	 have done so in a way that will not function properly with
 	 a weak symbol.  */
       else if (TREE_USED (olddecl)
-	       && TREE_SYMBOL_REFERENCED (DECL_ASSEMBLER_NAME (olddecl))
-	       /* ??? Not impossible that some platform may generate code
-		  that doesn't function *at all* with incorrect declaration
-		  before use.  However, all known at present will assume
-		  exteral and common symbols are not "local".  */
-	       /* ??? Probably these exceptions to the rule will just
-		  confuse users about the true nature of the problem and
-		  we should warn for *any* use before weakening.  */
-	       && ! (DECL_EXTERNAL (olddecl)
-		     || DECL_COMMON (olddecl)))
+	       && TREE_SYMBOL_REFERENCED (DECL_ASSEMBLER_NAME (olddecl)))
 	warning_with_decl (newdecl, "weak declaration of `%s' after first use results in unspecified behavior");
 
       if (SUPPORTS_WEAK)