From: jason <jason@138bc75d-0d04-0410-961f-82ee72b054a4>
Date: Tue, 21 Mar 2006 16:15:25 +0000 (+0000)
Subject:         PR c++/21581
X-Git-Url: http://git.sourceforge.jp/view?a=commitdiff_plain;h=30e9871167ca729023eb527698fb483394252e00;p=pf3gnuchains%2Fgcc-fork.git

    PR c++/21581
        * parser.c (cp_parser_declaration): Support attributes on
        anonymous namespaces.
        * name-lookup.c (push_namespace_with_attribs): Anonymous
        namespaces default to hidden visibility.


git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@112250 138bc75d-0d04-0410-961f-82ee72b054a4
---

diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog
index 983165551be..d754dd00de5 100644
--- a/gcc/cp/ChangeLog
+++ b/gcc/cp/ChangeLog
@@ -1,3 +1,11 @@
+2006-03-21  Jason Merrill  <jason@redhat.com>
+
+	PR c++/21581
+	* parser.c (cp_parser_declaration): Support attributes on
+	anonymous namespaces.
+	* name-lookup.c (push_namespace_with_attribs): Anonymous 
+	namespaces default to hidden visibility.
+
 2006-03-20  Jason Merrill  <jason@redhat.com>
 
 	PR c++/21764, c++/19238
diff --git a/gcc/cp/name-lookup.c b/gcc/cp/name-lookup.c
index 9b10fb4a9d7..4af73a30763 100644
--- a/gcc/cp/name-lookup.c
+++ b/gcc/cp/name-lookup.c
@@ -3046,7 +3046,7 @@ push_namespace_with_attribs (tree name, tree attributes)
       tree name = TREE_PURPOSE (d);
       tree args = TREE_VALUE (d);
       tree x;
-      
+
       if (! is_attribute_p ("visibility", name))
 	{
 	  warning (OPT_Wattributes, "%qs attribute directive ignored",
@@ -3055,16 +3055,25 @@ push_namespace_with_attribs (tree name, tree attributes)
 	}
 
       x = args ? TREE_VALUE (args) : NULL_TREE;
-      if (x == NULL_TREE || TREE_CODE (x) != STRING_CST)
+      if (x == NULL_TREE || TREE_CODE (x) != STRING_CST || TREE_CHAIN (args))
 	{
-	  warning (OPT_Wattributes, "%qs attribute requires an NTBS argument",
+	  warning (OPT_Wattributes, "%qs attribute requires a single NTBS argument",
 		   IDENTIFIER_POINTER (name));
 	  continue;
 	}
 
       current_binding_level->has_visibility = 1;
       push_visibility (TREE_STRING_POINTER (x));
+      goto found;
+    }
+  if (anon)
+    {
+      /* Anonymous namespaces default to hidden visibility.  This might
+	 change once we implement export.  */
+      current_binding_level->has_visibility = 1;
+      push_visibility ("hidden");
     }
+ found:
 #endif
 
   timevar_pop (TV_NAME_LOOKUP);
diff --git a/gcc/cp/parser.c b/gcc/cp/parser.c
index e04a8e7f0d6..c5e6b1781e6 100644
--- a/gcc/cp/parser.c
+++ b/gcc/cp/parser.c
@@ -7066,7 +7066,8 @@ cp_parser_declaration (cp_parser* parser)
 		&& (cp_lexer_peek_nth_token (parser->lexer, 3)->type
 		    != CPP_EQ))
 	       /* An unnamed namespace definition.  */
-	       || token2.type == CPP_OPEN_BRACE))
+	       || token2.type == CPP_OPEN_BRACE
+	       || token2.keyword == RID_ATTRIBUTE))
     cp_parser_namespace_definition (parser);
   /* Objective-C++ declaration/definition.  */
   else if (c_dialect_objc () && OBJC_IS_AT_KEYWORD (token1.keyword))
diff --git a/gcc/testsuite/g++.dg/ext/visibility/anon1.C b/gcc/testsuite/g++.dg/ext/visibility/anon1.C
new file mode 100644
index 00000000000..9a3d8a7f8ab
--- /dev/null
+++ b/gcc/testsuite/g++.dg/ext/visibility/anon1.C
@@ -0,0 +1,10 @@
+// PR c++/21581
+// Test for anonymous namespace default hidden visibility
+
+// { dg-require-visibility "" }
+// { dg-final { scan-hidden "_ZN.*1fEv" } }
+
+namespace
+{
+  int f() { }
+}