x86: Don't include <init-arch.h> in assembly codes
Commit Message
There is no need to include <init-arch.h> in assembly codes since all
x86 IFUNC selector functions are written in C. Tested on i686 and
x86-64. There is no code change in libc.so, ld.so and libmvec.so.
---
sysdeps/i386/i686/multiarch/bzero-ia32.S | 1 -
sysdeps/x86_64/fpu/multiarch/svml_d_sin8_core-avx2.S | 1 -
sysdeps/x86_64/fpu/multiarch/svml_s_expf16_core-avx2.S | 1 -
sysdeps/x86_64/multiarch/memset-sse2-unaligned-erms.S | 1 -
4 files changed, 4 deletions(-)
Comments
On 08/01/2018 05:20 PM, H.J. Lu wrote:
> There is no need to include <init-arch.h> in assembly codes since all
> x86 IFUNC selector functions are written in C. Tested on i686 and
> x86-64. There is no code change in libc.so, ld.so and libmvec.so.
> ---
> sysdeps/i386/i686/multiarch/bzero-ia32.S | 1 -
> sysdeps/x86_64/fpu/multiarch/svml_d_sin8_core-avx2.S | 1 -
> sysdeps/x86_64/fpu/multiarch/svml_s_expf16_core-avx2.S | 1 -
> sysdeps/x86_64/multiarch/memset-sse2-unaligned-erms.S | 1 -
> 4 files changed, 4 deletions(-)
Needs a ChangeLog entry. Actual change looks okay.
Thanks,
Florian
@@ -17,7 +17,6 @@
<http://www.gnu.org/licenses/>. */
#include <sysdep.h>
-#include <init-arch.h>
#if IS_IN (libc)
# define __bzero __bzero_ia32
@@ -17,7 +17,6 @@
<http://www.gnu.org/licenses/>. */
#include <sysdep.h>
-#include <init-arch.h>
#define _ZGVeN8v_sin _ZGVeN8v_sin_avx2_wrapper
#include "../svml_d_sin8_core.S"
@@ -17,7 +17,6 @@
<http://www.gnu.org/licenses/>. */
#include <sysdep.h>
-#include <init-arch.h>
#define _ZGVeN16v_expf _ZGVeN16v_expf_avx2_wrapper
#include "../svml_s_expf16_core.S"
@@ -19,7 +19,6 @@
#include <sysdep.h>
#include <shlib-compat.h>
-#include <init-arch.h>
#if IS_IN (libc)
# define MEMSET_SYMBOL(p,s) p##_sse2_##s