i386: Do not set internal_function

All calls to functions with the internal_function attribute
have been removed from assembler implementations, which means that
the definition of internal_function can be changed at the C level
without causing ABI issues with assembler code.

_dl_fixup still uses a regparm calling convention on i386, but this
is controlled through ARCH_FIXUP_ATTRIBUTE, not internal_function.

diff --git a/ChangeLog b/ChangeLog
index ac21eae7139dd0d27ae8174b49c69111d37e4ba3..2adb6bb29301240d043e51b1aed34f74ed527139 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2017-08-14  Florian Weimer  <fweimer@redhat.com>
+
+       i386: Do not set internal_function.
+       * config.h.in (USE_REGPARMS, internal_function): Remove.
+       * sysdeps/i386/configure.ac (USE_REGPARMS): Likewise.
+       * sysdeps/i386/configure (USE_REGPARMS): Likewise.
+
 2017-08-14  Florian Weimer  <fweimer@redhat.com>
 
        * elf/dl-init.c (_dl_init): Remove internal_function.

diff --git a/config.h.in b/config.h.in
index 22418576a014b19661edff6c6ee1117dd3e3d34b..014fb4ea0ef8ef5fde6cd8cea8fd6bcda3622d7a 100644 (file)
--- a/config.h.in
+++ b/config.h.in
@@ -47,10 +47,6 @@
 #undef STACK_PROTECTOR_LEVEL
 #endif
 
-/* Define if the regparm attribute shall be used for local functions
-   (gcc on ix86 only).  */
-#undef USE_REGPARMS
-
 /* Defined on SPARC if GCC emits GOTDATA relocations.  */
 #undef  HAVE_GCC_GOTDATA
 
@@ -100,12 +96,6 @@
 #undef HAVE_AARCH64_BE
 
 
-/* Defined to some form of __attribute__ ((...)) if the compiler supports
-   a different, more efficient calling convention.  */
-#if defined USE_REGPARMS && !defined PROF
-# define internal_function __attribute__ ((regparm (3), stdcall))
-#endif
-
 /* Linux specific: minimum supported kernel version.  */
 #undef __LINUX_KERNEL_VERSION
 

diff --git a/sysdeps/i386/configure b/sysdeps/i386/configure
index 5b55c5affe71ad3a1a144efb7f9ba0482a63ec6e..4d6685f7c2001c4dc85cbfe4e5b52455ac88f0ff 100644 (file)
--- a/sysdeps/i386/configure
+++ b/sysdeps/i386/configure
@@ -77,8 +77,5 @@ if test $libc_cv_asm_mpx = yes; then
 
 fi
 
-$as_echo "#define USE_REGPARMS 1" >>confdefs.h
-
-
 $as_echo "#define PI_STATIC_AND_HIDDEN 1" >>confdefs.h
 

diff --git a/sysdeps/i386/configure.ac b/sysdeps/i386/configure.ac
index 19ef33f34a334e0a3e438f6e2238755a08e08373..f7766ad2b8a33f4ecd4c450a12b421cba7cd1fc7 100644 (file)
--- a/sysdeps/i386/configure.ac
+++ b/sysdeps/i386/configure.ac
@@ -45,8 +45,6 @@ if test $libc_cv_asm_mpx = yes; then
   AC_DEFINE(HAVE_MPX_SUPPORT)
 fi
 
-AC_DEFINE(USE_REGPARMS)
-
 dnl It is always possible to access static and hidden symbols in an
 dnl position independent way.
 AC_DEFINE(PI_STATIC_AND_HIDDEN)