From 098a657fe449a217cf65c5270d5fbc8d40b5b4e6 Mon Sep 17 00:00:00 2001 From: Fangrui Song Date: Tue, 26 Apr 2022 09:26:22 -0700 Subject: elf: Replace PI_STATIC_AND_HIDDEN with opposite HIDDEN_VAR_NEEDS_DYNAMIC_RELOC PI_STATIC_AND_HIDDEN indicates whether accesses to internal linkage variables and hidden visibility variables in a shared object (ld.so) need dynamic relocations (usually R_*_RELATIVE). PI (position independent) in the macro name is a misnomer: a code sequence using GOT is typically position-independent as well, but using dynamic relocations does not meet the requirement. Not defining PI_STATIC_AND_HIDDEN is legacy and we expect that all new ports will define PI_STATIC_AND_HIDDEN. Current ports defining PI_STATIC_AND_HIDDEN are more than the opposite. Change the configure default. No functional change. Reviewed-by: Adhemerval Zanella --- sysdeps/x86/configure | 3 --- sysdeps/x86/configure.ac | 4 ---- 2 files changed, 7 deletions(-) (limited to 'sysdeps/x86') diff --git a/sysdeps/x86/configure b/sysdeps/x86/configure index 46d436fddf..a9c8c2ef67 100644 --- a/sysdeps/x86/configure +++ b/sysdeps/x86/configure @@ -190,8 +190,5 @@ fi config_vars="$config_vars enable-x86-isa-level = $libc_cv_include_x86_isa_level" -$as_echo "#define PI_STATIC_AND_HIDDEN 1" >>confdefs.h - - $as_echo "#define SUPPORT_STATIC_PIE 1" >>confdefs.h diff --git a/sysdeps/x86/configure.ac b/sysdeps/x86/configure.ac index 918b636891..edd43a2551 100644 --- a/sysdeps/x86/configure.ac +++ b/sysdeps/x86/configure.ac @@ -128,9 +128,5 @@ if test $libc_cv_include_x86_isa_level = yes; then fi LIBC_CONFIG_VAR([enable-x86-isa-level], [$libc_cv_include_x86_isa_level]) -dnl It is always possible to access static and hidden symbols in an -dnl position independent way. -AC_DEFINE(PI_STATIC_AND_HIDDEN) - dnl Static PIE is supported. AC_DEFINE(SUPPORT_STATIC_PIE) -- cgit 1.4.1