[Bug regression/52973] New: visibility attribute for class is not passed to its members

[Bug regression/52973] New: visibility attribute for class is not passed to its members

[holger.hopp at sap dot com]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52973

             Bug #: 52973
           Summary: visibility attribute for class is not passed to its
                    members
    Classification: Unclassified
           Product: gcc
           Version: 4.7.1
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: regression
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: holger.hopp@sap.com


In following code, gcc-4.7 does not accept the visibility attribute
defined in the class declaration:

template <class T> class __attribute__((visibility("default"))) a
{
public:
  /* A */ static int c;
};

class __attribute__((visibility("default"))) b : a <b> {};

template<> /* B */ int a<b>::c = 0;


With g++-4.7, a<b>::c is hidden, with g++-4.6 (and before) it is not.

$ g++-4.7 -fvisibility=hidden -c def.cpp && objdump -Ct def.o | grep "a<b>"
0000000000000000 g     O .bss    0000000000000004 .hidden a<b>::c

$ g++-4.6 -fvisibility=hidden -c def.cpp && objdump -Ct def.o | grep "a<b>"
0000000000000000 g     O .bss    0000000000000004 a<b>::c

Setting the default visibility at /* A */ or /* B */ works with
g++-4.7, but I think that it still should work also with the code above.

[Bug c++/52973] [4.7/4.8 Regression] visibility attribute for class is not passed to its members

[rguenth at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52973

Richard Guenther <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|regression                  |c++
      Known to work|                            |4.6.3
   Target Milestone|---                         |4.7.1
            Summary|visibility attribute for    |[4.7/4.8 Regression]
                   |class is not passed to its  |visibility attribute for
                   |members                     |class is not passed to its
                   |                            |members

[Bug c++/52973] [4.7/4.8 Regression] visibility attribute for class is not passed to its members

[rguenth at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52973

Richard Guenther <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P2

[Bug c++/52973] [4.7/4.8 Regression] visibility attribute for class is not passed to its members

[paolo.carlini at oracle dot com]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52973

Paolo Carlini <paolo.carlini at oracle dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jason at gcc dot gnu.org

--- Comment #1 from Paolo Carlini <paolo.carlini at oracle dot com> 2012-04-15 09:11:10 UTC ---
Jason, can you have a look?

[Bug c++/52973] [4.7/4.8 Regression] visibility attribute for class is not passed to its members

[jason at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52973

Jason Merrill <jason at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
   Last reconfirmed|                            |2012-06-01
         AssignedTo|unassigned at gcc dot       |jason at gcc dot gnu.org
                   |gnu.org                     |
     Ever Confirmed|0                           |1

[Bug c++/52973] [4.7/4.8 Regression] visibility attribute for class is not passed to its members

[jason at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52973

--- Comment #2 from Jason Merrill <jason at gcc dot gnu.org> 2012-06-01 21:36:35 UTC ---
The bug here is that we aren't applying the visibility attribute to b until
after we've decided that a<b> is hidden because b is hidden.  Changing the code
to

template <class T> class __attribute__((visibility("default"))) a
{
public:
  /* A */ static int c;
};

class __attribute__((visibility("default"))) b;
class b: a <b> {};

template<> /* B */ int a<b>::c = 0;

works around the issue.

[Bug c++/52973] [4.7/4.8 Regression] visibility attribute for class is not passed to its members

[jason at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52973

--- Comment #3 from Jason Merrill <jason at gcc dot gnu.org> 2012-06-02 04:03:57 UTC ---
Author: jason
Date: Sat Jun  2 04:03:51 2012
New Revision: 188137

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=188137
Log:
    PR c++/52973
    * parser.c (cp_parser_class_head): Apply attributes here.
    * semantics.c (begin_class_definition): Not here.
    * cp-tree.h: Adjust.

Added:
    trunk/gcc/testsuite/g++.dg/ext/visibility/template12.C
Modified:
    trunk/gcc/cp/ChangeLog
    trunk/gcc/cp/cp-tree.h
    trunk/gcc/cp/parser.c
    trunk/gcc/cp/semantics.c
    trunk/gcc/objcp/ChangeLog
    trunk/gcc/objcp/objcp-decl.c
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/testsuite/g++.dg/ext/attrib14.C

[Bug c++/52973] [4.7/4.8 Regression] visibility attribute for class is not passed to its members

[jason at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52973

--- Comment #4 from Jason Merrill <jason at gcc dot gnu.org> 2012-06-02 04:05:34 UTC ---
Author: jason
Date: Sat Jun  2 04:05:25 2012
New Revision: 188138

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=188138
Log:
    PR c++/52973
    * parser.c (cp_parser_class_head): Apply attributes here.
    * semantics.c (begin_class_definition): Not here.
    * cp-tree.h: Adjust.

Added:
    branches/gcc-4_7-branch/gcc/testsuite/g++.dg/ext/visibility/template12.C
Modified:
    branches/gcc-4_7-branch/gcc/cp/ChangeLog
    branches/gcc-4_7-branch/gcc/cp/cp-tree.h
    branches/gcc-4_7-branch/gcc/cp/parser.c
    branches/gcc-4_7-branch/gcc/cp/semantics.c
    branches/gcc-4_7-branch/gcc/objcp/objcp-decl.c
    branches/gcc-4_7-branch/gcc/testsuite/ChangeLog
    branches/gcc-4_7-branch/gcc/testsuite/g++.dg/ext/attrib14.C

[Bug c++/52973] [4.7/4.8 Regression] visibility attribute for class is not passed to its members

[jason at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52973

Jason Merrill <jason at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED

--- Comment #5 from Jason Merrill <jason at gcc dot gnu.org> 2012-06-02 04:06:08 UTC ---
Fixed for 4.7.1.

[Bug c++/52973] [4.7/4.8 Regression] visibility attribute for class is not passed to its members

[jason at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52973

Jason Merrill <jason at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED

--- Comment #4 from Jason Merrill <jason at gcc dot gnu.org> 2012-06-02 04:05:34 UTC ---
Author: jason
Date: Sat Jun  2 04:05:25 2012
New Revision: 188138

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=188138
Log:
    PR c++/52973
    * parser.c (cp_parser_class_head): Apply attributes here.
    * semantics.c (begin_class_definition): Not here.
    * cp-tree.h: Adjust.

Added:
    branches/gcc-4_7-branch/gcc/testsuite/g++.dg/ext/visibility/template12.C
Modified:
    branches/gcc-4_7-branch/gcc/cp/ChangeLog
    branches/gcc-4_7-branch/gcc/cp/cp-tree.h
    branches/gcc-4_7-branch/gcc/cp/parser.c
    branches/gcc-4_7-branch/gcc/cp/semantics.c
    branches/gcc-4_7-branch/gcc/objcp/objcp-decl.c
    branches/gcc-4_7-branch/gcc/testsuite/ChangeLog
    branches/gcc-4_7-branch/gcc/testsuite/g++.dg/ext/attrib14.C

--- Comment #5 from Jason Merrill <jason at gcc dot gnu.org> 2012-06-02 04:06:08 UTC ---
Fixed for 4.7.1.