RepoMirrors/musl
1
0
Fork 0
mirror of git://git.musl-libc.org/musl synced 2025-01-31 02:51:32 +00:00
Code Issues Packages Projects Releases Wiki Activity

in visibility preinclude, remove overrides for stdin/stdout/stderr

Browse Source 
the motivation for this change is that the extra declaration (with or
without visibility) using "struct _IO_FILE" instead of "FILE" seems to
trigger a bug in gcc 3.x where it considers the types mismatched.
however, this change also results in slightly better code and it is
valid because (1) these three objects are constant, and (2) applying
the & operator to any of them is invalid C, since they are not even
specified to be objects. thus it does not matter if the application
and libc see different addresses for them, as long as the (initial,
unchanging) value is seen the same by both.
This commit is contained in:
Rich Felker 2015-04-22 03:07:38 -04:00
parent f203b3511f
commit b8dda24fe1
1 changed files with 0 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/internal/vis.h
View File

@ -11,9 +11,6 @@
* fail to support copy relocations for protected data, so exclude all
* exported data symbols. */
__attribute__((__visibility__("default")))
extern struct _IO_FILE *const stdin, *const stdout, *const stderr;
__attribute__((__visibility__("default")))
extern int optind, opterr, optopt, optreset, __optreset, getdate_err, h_errno, daylight, __daylight, signgam, __signgam;