From 9b95fd0944e4206949e90633c3fed088202810ec Mon Sep 17 00:00:00 2001 From: Rich Felker Date: Wed, 5 Sep 2018 12:43:34 -0400 Subject: define and use internal macros for hidden visibility, weak refs this cleans up what had become widespread direct inline use of "GNU C" style attributes directly in the source, and lowers the barrier to increased use of hidden visibility, which will be useful to recovering some of the efficiency lost when the protected visibility hack was dropped in commit dc2f368e565c37728b0d620380b849c3a1ddd78f, especially on archs where the PLT ABI is costly. --- src/env/__init_tls.c | 3 +-- src/env/__libc_start_main.c | 3 +-- src/env/__stack_chk_fail.c | 3 +-- 3 files changed, 3 insertions(+), 6 deletions(-) (limited to 'src/env') diff --git a/src/env/__init_tls.c b/src/env/__init_tls.c index 31d324a8..e0224243 100644 --- a/src/env/__init_tls.c +++ b/src/env/__init_tls.c @@ -72,8 +72,7 @@ typedef Elf32_Phdr Phdr; typedef Elf64_Phdr Phdr; #endif -__attribute__((__weak__, __visibility__("hidden"))) -extern const size_t _DYNAMIC[]; +extern weak hidden const size_t _DYNAMIC[]; static void static_init_tls(size_t *aux) { diff --git a/src/env/__libc_start_main.c b/src/env/__libc_start_main.c index 0583f686..c1b06697 100644 --- a/src/env/__libc_start_main.c +++ b/src/env/__libc_start_main.c @@ -11,8 +11,7 @@ void __init_tls(size_t *); static void dummy(void) {} weak_alias(dummy, _init); -__attribute__((__weak__, __visibility__("hidden"))) -extern void (*const __init_array_start)(void), (*const __init_array_end)(void); +extern weak hidden void (*const __init_array_start)(void), (*const __init_array_end)(void); static void dummy1(void *p) {} weak_alias(dummy1, __init_ssp); diff --git a/src/env/__stack_chk_fail.c b/src/env/__stack_chk_fail.c index 4de82fd9..e32596d1 100644 --- a/src/env/__stack_chk_fail.c +++ b/src/env/__stack_chk_fail.c @@ -17,7 +17,6 @@ void __stack_chk_fail(void) a_crash(); } -__attribute__((__visibility__("hidden"))) -void __stack_chk_fail_local(void); +hidden void __stack_chk_fail_local(void); weak_alias(__stack_chk_fail, __stack_chk_fail_local); -- cgit v1.2.1