AArch64: Check for SVE in ifuncs [BZ #28744]
Checks
| Context |
Check |
Description |
| dj/TryBot-apply_patch |
success
|
Patch applied to master at the time it was sent
|
| dj/TryBot-32bit |
success
|
Build for i686
|
Commit Message
Add a check for SVE in the A64FX ifuncs for memcpy, memset and memmove.
This fixes BZ #28744, OK for commit?
---
Comments
The 01/05/2022 16:31, Wilco Dijkstra via Libc-alpha wrote:
> Add a check for SVE in the A64FX ifuncs for memcpy, memset and memmove.
> This fixes BZ #28744, OK for commit?
>
This is OK, thanks.
> ---
> diff --git a/sysdeps/aarch64/multiarch/memcpy.c b/sysdeps/aarch64/multiarch/memcpy.c
> index 7dac7b76e475ba07db7b3bee58b642921b51b654..a476dd548fd04fddcd0b39cc3263866769bb309a 100644
> --- a/sysdeps/aarch64/multiarch/memcpy.c
> +++ b/sysdeps/aarch64/multiarch/memcpy.c
> @@ -48,7 +48,7 @@ libc_ifunc (__libc_memcpy,
> || IS_NEOVERSE_V1 (midr)
> ? __memcpy_simd
> # if HAVE_AARCH64_SVE_ASM
> - : (IS_A64FX (midr)
> + : (IS_A64FX (midr) && sve
> ? __memcpy_a64fx
> : __memcpy_generic))))));
> # else
> diff --git a/sysdeps/aarch64/multiarch/memmove.c b/sysdeps/aarch64/multiarch/memmove.c
> index 48f8e4616644a0f29718357687445ed4002b726c..4f7d7eedfd943d1b6a54e90dc973406b30cc8db9 100644
> --- a/sysdeps/aarch64/multiarch/memmove.c
> +++ b/sysdeps/aarch64/multiarch/memmove.c
> @@ -48,7 +48,7 @@ libc_ifunc (__libc_memmove,
> || IS_NEOVERSE_V1 (midr)
> ? __memmove_simd
> # if HAVE_AARCH64_SVE_ASM
> - : (IS_A64FX (midr)
> + : (IS_A64FX (midr) && sve
> ? __memmove_a64fx
> : __memmove_generic))))));
> # else
> diff --git a/sysdeps/aarch64/multiarch/memset.c b/sysdeps/aarch64/multiarch/memset.c
> index 3692b079e3bfaa4fa03dda7f5c2bef3486e8e05f..c4008f346b34ec5b4b767a1433cf72692572a714 100644
> --- a/sysdeps/aarch64/multiarch/memset.c
> +++ b/sysdeps/aarch64/multiarch/memset.c
> @@ -44,7 +44,7 @@ libc_ifunc (__libc_memset,
> : (IS_EMAG (midr) && zva_size == 64
> ? __memset_emag
> # if HAVE_AARCH64_SVE_ASM
> - : (IS_A64FX (midr)
> + : (IS_A64FX (midr) && sve
> ? __memset_a64fx
> : __memset_generic))));
> # else
>
@@ -48,7 +48,7 @@ libc_ifunc (__libc_memcpy,
|| IS_NEOVERSE_V1 (midr)
? __memcpy_simd
# if HAVE_AARCH64_SVE_ASM
- : (IS_A64FX (midr)
+ : (IS_A64FX (midr) && sve
? __memcpy_a64fx
: __memcpy_generic))))));
# else
@@ -48,7 +48,7 @@ libc_ifunc (__libc_memmove,
|| IS_NEOVERSE_V1 (midr)
? __memmove_simd
# if HAVE_AARCH64_SVE_ASM
- : (IS_A64FX (midr)
+ : (IS_A64FX (midr) && sve
? __memmove_a64fx
: __memmove_generic))))));
# else
@@ -44,7 +44,7 @@ libc_ifunc (__libc_memset,
: (IS_EMAG (midr) && zva_size == 64
? __memset_emag
# if HAVE_AARCH64_SVE_ASM
- : (IS_A64FX (midr)
+ : (IS_A64FX (midr) && sve
? __memset_a64fx
: __memset_generic))));
# else