diff mbox

[powerpc] Fix tiny bug in strncmp.c

Message ID ad7190f2-a310-ae7e-8be3-7b460d9b372c@us.ibm.com
State Committed
Headers

Commit Message

Paul A. Clarke Jan. 15, 2019, 6:42 p.m. UTC 
  A single underscore was omitted in
sysdeps/powerpc/powerpc64/multiarch/strncmp.c, resulting in use of
power8 version of strncmp instead of power9 version, with significant
performance degradation.

2019-01-15  Paul A. Clarke  <pc@us.ibm.com>

	* sysdeps/powerpc/powerpc64/multiarch/strncmp.c: Fix #ifdef.

Comments

Adhemerval Zanella Jan. 15, 2019, 7:21 p.m. UTC | #1 
On 15/01/2019 16:42, Paul Clarke wrote:
> A single underscore was omitted in
> sysdeps/powerpc/powerpc64/multiarch/strncmp.c, resulting in use of
> power8 version of strncmp instead of power9 version, with significant
> performance degradation.
> 
> 2019-01-15  Paul A. Clarke  <pc@us.ibm.com>
> 
> 	* sysdeps/powerpc/powerpc64/multiarch/strncmp.c: Fix #ifdef.
> 
> diff --git a/sysdeps/powerpc/powerpc64/multiarch/strncmp.c b/sysdeps/powerpc/powerpc64/multiarch/strncmp.c
> index ab7379d..d2791ce 100644
> --- a/sysdeps/powerpc/powerpc64/multiarch/strncmp.c
> +++ b/sysdeps/powerpc/powerpc64/multiarch/strncmp.c
> @@ -37,7 +37,7 @@ extern __typeof (strncmp) __strncmp_power9 attribute_hidden;
>  /* Avoid DWARF definition DIE on ifunc symbol so that GDB can handle
>     ifunc symbol properly.  */
>  libc_ifunc_redirected (__redirect_strncmp, strncmp,
> -# ifdef __LITTLE_ENDIAN_
> +# ifdef __LITTLE_ENDIAN__
>  			(hwcap2 & PPC_FEATURE2_ARCH_3_00)
>  			? __strncmp_power9 :
>  # endif
> 

LGTM, thanks.
Tulio Magno Quites Machado Filho Jan. 16, 2019, 4:03 p.m. UTC | #2 
Paul Clarke <pc@us.ibm.com> writes:

> A single underscore was omitted in
> sysdeps/powerpc/powerpc64/multiarch/strncmp.c, resulting in use of
> power8 version of strncmp instead of power9 version, with significant
> performance degradation.
>
> 2019-01-15  Paul A. Clarke  <pc@us.ibm.com>
>
> 	* sysdeps/powerpc/powerpc64/multiarch/strncmp.c: Fix #ifdef.

LGTM too.

Pushed as 008b598e2a495024f9777006716cfd8668f3db33.

Thanks!
diff mbox

Patch

diff --git a/sysdeps/powerpc/powerpc64/multiarch/strncmp.c b/sysdeps/powerpc/powerpc64/multiarch/strncmp.c
index ab7379d..d2791ce 100644
--- a/sysdeps/powerpc/powerpc64/multiarch/strncmp.c
+++ b/sysdeps/powerpc/powerpc64/multiarch/strncmp.c
@@ -37,7 +37,7 @@  extern __typeof (strncmp) __strncmp_power9 attribute_hidden;
 /* Avoid DWARF definition DIE on ifunc symbol so that GDB can handle
    ifunc symbol properly.  */
 libc_ifunc_redirected (__redirect_strncmp, strncmp,
-# ifdef __LITTLE_ENDIAN_
+# ifdef __LITTLE_ENDIAN__
 			(hwcap2 & PPC_FEATURE2_ARCH_3_00)
 			? __strncmp_power9 :
 # endif