diff options
| -rw-r--r-- | gcc/cp/ChangeLog | 5 | ||||
| -rw-r--r-- | gcc/cp/decl2.c | 9 |
2 files changed, 11 insertions, 3 deletions
diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog index 9383ccd..22afe19 100644 --- a/gcc/cp/ChangeLog +++ b/gcc/cp/ChangeLog @@ -1,4 +1,9 @@ 2008-01-25 Jason Merrill <jason@redhat.com> + + * decl2.c (is_late_template_attribute): Don't defer attribute + visibility just because the type is dependent. + +2008-01-25 Jason Merrill <jason@redhat.com> Mark Mitchell <mark@codesourcery.com> PR c++/31780 diff --git a/gcc/cp/decl2.c b/gcc/cp/decl2.c index 550d0bc..fa8d28a 100644 --- a/gcc/cp/decl2.c +++ b/gcc/cp/decl2.c @@ -1014,9 +1014,12 @@ is_late_template_attribute (tree attr, tree decl) || code == BOUND_TEMPLATE_TEMPLATE_PARM || code == TYPENAME_TYPE) return true; - /* Also defer attributes on dependent types. This is not necessary - in all cases, but is the better default. */ - else if (dependent_type_p (type)) + /* Also defer most attributes on dependent types. This is not + necessary in all cases, but is the better default. */ + else if (dependent_type_p (type) + /* But attribute visibility specifically works on + templates. */ + && !is_attribute_p ("visibility", name)) return true; else return false; |