re PR c/7425 (deprecated attribute doesn't merge with multiple declarations)

	PR c/7425
	* c-decl.c (merge_decls): Merge TREE_DEPRECATED.

cp:
	* decl.c (duplicate_decls): Merge TREE_DEPRECATED.

testsuite:
	* gcc.dg/deprecated-3.c, g++.dg/warn/deprecated-2.C: New tests.

From-SVN: r88303

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index ef37139c795d9ebe48ee50feb1abdecd3a06b8d9..cf4671908c08bc1786c00ef2f4fd67bc4ac29553 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,8 @@
+2004-09-29  Joseph S. Myers  <jsm@polyomino.org.uk>
+
+       PR c/7425
+       * c-decl.c (merge_decls): Merge TREE_DEPRECATED.
+
 2004-09-29  Eric Christopher  <echristo@redhat.com>
 
        * fold-const.c (tree_swap_operands_p): Remove duplicated code.

diff --git a/gcc/c-decl.c b/gcc/c-decl.c
index 413787633dcbe31e200cc5bcc7bdcd2153999323..38a13fd201ccc855e5a4d8dc85cbc817a95e069a 100644 (file)
--- a/gcc/c-decl.c
+++ b/gcc/c-decl.c
@@ -1586,6 +1586,10 @@ merge_decls (tree newdecl, tree olddecl, tree newtype, tree oldtype)
        make_var_volatile (newdecl);
     }
 
+  /* Merge deprecatedness.  */
+  if (TREE_DEPRECATED (newdecl))
+    TREE_DEPRECATED (olddecl) = 1;
+
   /* Keep source location of definition rather than declaration.  */
   if (DECL_INITIAL (newdecl) == 0 && DECL_INITIAL (olddecl) != 0)
     DECL_SOURCE_LOCATION (newdecl) = DECL_SOURCE_LOCATION (olddecl);

diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog
index 4933e07af124a384d11462898486e77d720eb48f..fa42fad9a439ff213ea6e209b8734ea4ab65ff0c 100644 (file)
--- a/gcc/cp/ChangeLog
+++ b/gcc/cp/ChangeLog
@@ -1,3 +1,7 @@
+2004-09-29  Joseph S. Myers  <jsm@polyomino.org.uk>
+
+       * decl.c (duplicate_decls): Merge TREE_DEPRECATED.
+
 2004-09-29  Jason Merrill  <jason@redhat.com>
 
        PR tree-optimization/17697

diff --git a/gcc/cp/decl.c b/gcc/cp/decl.c
index 3e394ab43e8bb3d6ac8346ee275c312a67eb3d8a..c27d041378b819ef3cefc1c1a46906954f7861ce 100644 (file)
--- a/gcc/cp/decl.c
+++ b/gcc/cp/decl.c
@@ -1722,6 +1722,10 @@ duplicate_decls (tree newdecl, tree olddecl)
       if (TREE_NOTHROW (newdecl))
        TREE_NOTHROW (olddecl) = 1;
 
+      /* Merge deprecatedness.  */
+      if (TREE_DEPRECATED (newdecl))
+       TREE_DEPRECATED (olddecl) = 1;
+
       /* Merge the initialization information.  */
       if (DECL_INITIAL (newdecl) == NULL_TREE
          && DECL_INITIAL (olddecl) != NULL_TREE)

diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 05c6e3f4cbce66c7aa1575b3eea573aaf8b7db2c..13e29d9a153e8749f404418d51a1284acf020d18 100644 (file)
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2004-09-29  Joseph S. Myers  <jsm@polyomino.org.uk>
+
+       PR c/7425
+       * gcc.dg/deprecated-3.c, g++.dg/warn/deprecated-2.C: New tests.
+
 2004-09-29  Jakub Jelinek  <jakub@redhat.com>
 
        * gcc.c-torture/execute/stdarg-1.c: New test.

diff --git a/gcc/testsuite/g++.dg/warn/deprecated-2.C b/gcc/testsuite/g++.dg/warn/deprecated-2.C
new file mode 100644 (file)
index 0000000..afddc8f
--- /dev/null
+++ b/gcc/testsuite/g++.dg/warn/deprecated-2.C
@@ -0,0 +1,11 @@
+/* Test __attribute__((deprecated)).  Test merging with multiple
+   declarations.  Bug 7425 (C++ version).  */
+/* { dg-do compile } */
+/* { dg-options "" } */
+
+void func(void);
+void func(void) __attribute__((deprecated));
+
+void f(void) {
+  func(); /* { dg-warning "'func' is deprecated" } */
+}

diff --git a/gcc/testsuite/gcc.dg/deprecated-3.c b/gcc/testsuite/gcc.dg/deprecated-3.c
new file mode 100644 (file)
index 0000000..2c35bb3
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/deprecated-3.c
@@ -0,0 +1,11 @@
+/* Test __attribute__((deprecated)).  Test merging with multiple
+   declarations.  Bug 7425.  */
+/* { dg-do compile } */
+/* { dg-options "" } */
+
+void func(void);
+void func(void) __attribute__((deprecated));
+
+void f(void) {
+  func(); /* { dg-warning "'func' is deprecated" } */
+}