cleanup and work around visibility bug in gcc 3 that affects x86_64

in gcc 3, the visibility attribute must be placed on both the
declaration and on the definition. if it's omitted from the
definition, the compiler fails to emit the ".hidden" directive in the
assembly, and the linker will either generate textrels (if supported,
such as on i386) or refuse to link (on targets where certain types of
textrels are forbidden or impossible without further assumptions about
memory layout, such as on x86_64).

this patch also unifies the decision about when to use visibility into
libc.h and makes the visibility in the utf-8 state machine tables
based on libc.h rather than a duplicate test.
This commit is contained in:
Rich Felker
2012-02-23 21:24:56 -05:00
parent 00b883a955
commit bae2e52bfd
4 changed files with 15 additions and 12 deletions

View File

@ -7,5 +7,5 @@ struct __libc *__libc_loc()
return &__libc;
}
#else
struct __libc __libc;
struct __libc __libc ATTR_LIBC_VISIBILITY;
#endif