diff options
author | Raoni Fassina Firmino <raoni@linux.ibm.com> | 2021-05-03 16:59:35 -0300 |
---|---|---|
committer | Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com> | 2021-05-07 15:52:23 -0300 |
commit | 17a73a6d8b4c46f3e87fc53c7c25fa7cec01d707 (patch) | |
tree | 6f8afcb3bf82fe13ff6f3aad2d3d56e4f32f8650 /sysdeps/powerpc/powerpc64/multiarch/ifunc-impl-list.c | |
parent | 310be3cc09d5b1471174bc12f1860169e686bd65 (diff) | |
download | glibc-17a73a6d8b4c46f3e87fc53c7c25fa7cec01d707.tar glibc-17a73a6d8b4c46f3e87fc53c7c25fa7cec01d707.tar.gz glibc-17a73a6d8b4c46f3e87fc53c7c25fa7cec01d707.tar.bz2 glibc-17a73a6d8b4c46f3e87fc53c7c25fa7cec01d707.zip |
powerpc64le: Fix ifunc selection for memset, memmove, bzero and bcopy
The hwcap2 check for the aforementioned functions should check for
both PPC_FEATURE2_ARCH_3_1 and PPC_FEATURE2_HAS_ISEL but was
mistakenly checking for any one of them, enabling isa 3.1 version of
the functions in incompatible processors, like POWER8.
Reviewed-by: Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com>
Diffstat (limited to 'sysdeps/powerpc/powerpc64/multiarch/ifunc-impl-list.c')
-rw-r--r-- | sysdeps/powerpc/powerpc64/multiarch/ifunc-impl-list.c | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/sysdeps/powerpc/powerpc64/multiarch/ifunc-impl-list.c b/sysdeps/powerpc/powerpc64/multiarch/ifunc-impl-list.c index 49d9a33e65..b123c6a3d3 100644 --- a/sysdeps/powerpc/powerpc64/multiarch/ifunc-impl-list.c +++ b/sysdeps/powerpc/powerpc64/multiarch/ifunc-impl-list.c @@ -75,9 +75,9 @@ __libc_ifunc_impl_list (const char *name, struct libc_ifunc_impl *array, IFUNC_IMPL (i, name, memmove, #ifdef __LITTLE_ENDIAN__ IFUNC_IMPL_ADD (array, i, memmove, - hwcap2 & (PPC_FEATURE2_ARCH_3_1 | - PPC_FEATURE2_HAS_ISEL) - && (hwcap & PPC_FEATURE_HAS_VSX), + hwcap2 & PPC_FEATURE2_ARCH_3_1 + && hwcap2 & PPC_FEATURE2_HAS_ISEL + && hwcap & PPC_FEATURE_HAS_VSX, __memmove_power10) #endif IFUNC_IMPL_ADD (array, i, memmove, hwcap & PPC_FEATURE_HAS_VSX, @@ -88,8 +88,8 @@ __libc_ifunc_impl_list (const char *name, struct libc_ifunc_impl *array, IFUNC_IMPL (i, name, memset, #ifdef __LITTLE_ENDIAN__ IFUNC_IMPL_ADD (array, i, memset, - hwcap2 & (PPC_FEATURE2_ARCH_3_1 | - PPC_FEATURE2_HAS_ISEL) + hwcap2 & PPC_FEATURE2_ARCH_3_1 + && hwcap2 & PPC_FEATURE2_HAS_ISEL && hwcap & PPC_FEATURE_HAS_VSX, __memset_power10) #endif @@ -196,8 +196,8 @@ __libc_ifunc_impl_list (const char *name, struct libc_ifunc_impl *array, IFUNC_IMPL (i, name, bzero, #ifdef __LITTLE_ENDIAN__ IFUNC_IMPL_ADD (array, i, bzero, - hwcap2 & (PPC_FEATURE2_ARCH_3_1 | - PPC_FEATURE2_HAS_ISEL) + hwcap2 & PPC_FEATURE2_ARCH_3_1 + && hwcap2 & PPC_FEATURE2_HAS_ISEL && hwcap & PPC_FEATURE_HAS_VSX, __bzero_power10) #endif @@ -215,9 +215,9 @@ __libc_ifunc_impl_list (const char *name, struct libc_ifunc_impl *array, IFUNC_IMPL (i, name, bcopy, #ifdef __LITTLE_ENDIAN__ IFUNC_IMPL_ADD (array, i, bcopy, - hwcap2 & (PPC_FEATURE2_ARCH_3_1 | - PPC_FEATURE2_HAS_ISEL) - && (hwcap & PPC_FEATURE_HAS_VSX), + hwcap2 & PPC_FEATURE2_ARCH_3_1 + && hwcap2 & PPC_FEATURE2_HAS_ISEL + && hwcap & PPC_FEATURE_HAS_VSX, __bcopy_power10) #endif IFUNC_IMPL_ADD (array, i, bcopy, hwcap & PPC_FEATURE_HAS_VSX, |