re PR c++/46803 (libstdc++ build errors on invalid OpenBSD system header attributes)

	PR c++/46803
	* c-common.c (attribute_takes_identifier_p): Assume that an
	unknown attribute takes an identifier.

From-SVN: r170885

diff --git a/gcc/c-family/ChangeLog b/gcc/c-family/ChangeLog
index 60fc08fa8492f2b0254a97dda356704ab22fb900..35ca9a3767a53c95743e2dc63d5fc354a8ac2e01 100644 (file)
--- a/gcc/c-family/ChangeLog
+++ b/gcc/c-family/ChangeLog
@@ -1,3 +1,9 @@
+2011-03-11  Jason Merrill  <jason@redhat.com>
+
+       PR c++/46803
+       * c-common.c (attribute_takes_identifier_p): Assume that an
+       unknown attribute takes an identifier.
+
 2011-03-07  Nathan Froyd  <froydnj@codesourcery.com>
 
        PR c/47786

diff --git a/gcc/c-family/c-common.c b/gcc/c-family/c-common.c
index f0296612e5193eb9559417f5f77c9d0ca7163c09..32b9a70d72d852911fdd47cc1508139be01c044a 100644 (file)
--- a/gcc/c-family/c-common.c
+++ b/gcc/c-family/c-common.c
@@ -5665,9 +5665,14 @@ c_init_attributes (void)
 bool
 attribute_takes_identifier_p (const_tree attr_id)
 {
-  if (is_attribute_p ("mode", attr_id)
-      || is_attribute_p ("format", attr_id)
-      || is_attribute_p ("cleanup", attr_id))
+  struct attribute_spec *spec = lookup_attribute_spec (attr_id);
+  if (spec == NULL)
+    /* Unknown attribute that we'll end up ignoring, return true so we
+       don't complain about an identifier argument.  */
+    return true;
+  else if (!strcmp ("mode", spec->name)
+          || !strcmp ("format", spec->name)
+          || !strcmp ("cleanup", spec->name))
     return true;
   else
     return targetm.attribute_takes_identifier_p (attr_id);

diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index cf7d6d8a7cb906374bc25e3910f5b5cf3bb0daad..6734535f7bbb1ac73883609fd219e3ee266f399d 100644 (file)
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,5 +1,7 @@
 2011-03-11  Jason Merrill  <jason@redhat.com>
 
+       * g++.dg/ext/attrib40.C: New.
+
        * g++.dg/cpp0x/regress/array1.C: New.
 
 2011-03-11  Richard Guenther  <rguenther@suse.de>

diff --git a/gcc/testsuite/g++.dg/ext/attrib40.C b/gcc/testsuite/g++.dg/ext/attrib40.C
new file mode 100644 (file)
index 0000000..9c3f761
--- /dev/null
+++ b/gcc/testsuite/g++.dg/ext/attrib40.C
@@ -0,0 +1,4 @@
+// PR c++/46803
+
+int strftime(char *, int, const char *, const struct tm *)
+        __attribute__ ((__bounded__(__string__,1,2))); // { dg-warning "ignored" }