diff mbox series

[1/2] x86-64: Fix strcmp-avx2.S

Message ID 20220204191200.73982-1-hjl.tools@gmail.com
State New
Headers show
Series [1/2] x86-64: Fix strcmp-avx2.S | expand

Commit Message

H.J. Lu Feb. 4, 2022, 7:11 p.m. UTC
Change "movl %edx, %rdx" to "movl %edx, %edx" in:

commit b77b06e0e296f1a2276c27a67e1d44f2cfa38d45
Author: Noah Goldstein <goldstein.w.n@gmail.com>
Date:   Mon Jan 10 15:35:38 2022 -0600

    x86: Optimize strcmp-avx2.S
---
 sysdeps/x86_64/multiarch/strcmp-avx2.S | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Sunil Pandey May 4, 2022, 5:42 a.m. UTC | #1
On Fri, Feb 4, 2022 at 11:12 AM H.J. Lu via Libc-alpha
<libc-alpha@sourceware.org> wrote:
>
> Change "movl %edx, %rdx" to "movl %edx, %edx" in:
>
> commit b77b06e0e296f1a2276c27a67e1d44f2cfa38d45
> Author: Noah Goldstein <goldstein.w.n@gmail.com>
> Date:   Mon Jan 10 15:35:38 2022 -0600
>
>     x86: Optimize strcmp-avx2.S
> ---
>  sysdeps/x86_64/multiarch/strcmp-avx2.S | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/sysdeps/x86_64/multiarch/strcmp-avx2.S b/sysdeps/x86_64/multiarch/strcmp-avx2.S
> index 28d6a0025a..99e5349be8 100644
> --- a/sysdeps/x86_64/multiarch/strcmp-avx2.S
> +++ b/sysdeps/x86_64/multiarch/strcmp-avx2.S
> @@ -106,7 +106,7 @@ ENTRY(STRCMP)
>  # ifdef USE_AS_STRNCMP
>  #  ifdef __ILP32__
>         /* Clear the upper 32 bits.  */
> -       movl    %edx, %rdx
> +       movl    %edx, %edx
>  #  endif
>         cmp     $1, %RDX_LP
>         /* Signed comparison intentional. We use this branch to also
> --
> 2.34.1
>

I would like to backport this patch to release branches.
Any comments or objections?

--Sunil
diff mbox series

Patch

diff --git a/sysdeps/x86_64/multiarch/strcmp-avx2.S b/sysdeps/x86_64/multiarch/strcmp-avx2.S
index 28d6a0025a..99e5349be8 100644
--- a/sysdeps/x86_64/multiarch/strcmp-avx2.S
+++ b/sysdeps/x86_64/multiarch/strcmp-avx2.S
@@ -106,7 +106,7 @@  ENTRY(STRCMP)
 # ifdef USE_AS_STRNCMP
 #  ifdef __ILP32__
 	/* Clear the upper 32 bits.  */
-	movl	%edx, %rdx
+	movl	%edx, %edx
 #  endif
 	cmp	$1, %RDX_LP
 	/* Signed comparison intentional. We use this branch to also