[Bug string/29839] New: [2.37 Regression] Wrong result in function __strncpy_avx2: 75 != 0

[Bug string/29839] New: [2.37 Regression] Wrong result in function __strncpy_avx2: 75 != 0

[hjl.tools at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29839

            Bug ID: 29839
           Summary: [2.37 Regression] Wrong result in function
                    __strncpy_avx2: 75 != 0
           Product: glibc
           Version: 2.37
            Status: NEW
          Severity: normal
          Priority: P2
         Component: string
          Assignee: unassigned at sourceware dot org
          Reporter: hjl.tools at gmail dot com
                CC: goldstein.w.n at gmail dot com
  Target Milestone: ---
            Target: x32

987f8647e8 scripts: Add "|" operator support to glibcpp's parsing

gave

string/tst-size_t-strncpy: Wrong result in function __strncpy_avx2: 75 != 0

on x32.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug string/29839] [2.37 Regression] Wrong result in function __strncpy_avx2: 75 != 0

[hjl.tools at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29839

--- Comment #1 from H.J. Lu <hjl.tools at gmail dot com> ---
I am testing this:

diff --git a/sysdeps/x86_64/multiarch/strncpy-avx2.S
b/sysdeps/x86_64/multiarch/strncpy-avx2.S
index e9afd8fbed..3e6350ce4a 100644
--- a/sysdeps/x86_64/multiarch/strncpy-avx2.S
+++ b/sysdeps/x86_64/multiarch/strncpy-avx2.S
@@ -52,6 +52,10 @@

   .section SECTION(.text), "ax", @progbits
 ENTRY(STRNCPY)
+# ifdef __ILP32__
+  /* Clear the upper 32 bits.  */
+  movl  %edx, %edx
+# endif
   /* Filter zero length strings and very long strings.  Zero
      length strings just return, very long strings are handled by
      just running rep stos{b|l} to zero set (which will almost

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug string/29839] [2.37 Regression] Wrong result in function __strncpy_avx2: 75 != 0

[goldstein.w.n at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29839

--- Comment #2 from Noah Goldstein <goldstein.w.n at gmail dot com> ---
(In reply to H.J. Lu from comment #1)
> I am testing this:
> 
> diff --git a/sysdeps/x86_64/multiarch/strncpy-avx2.S
> b/sysdeps/x86_64/multiarch/strncpy-avx2.S
> index e9afd8fbed..3e6350ce4a 100644
> --- a/sysdeps/x86_64/multiarch/strncpy-avx2.S
> +++ b/sysdeps/x86_64/multiarch/strncpy-avx2.S
> @@ -52,6 +52,10 @@
>  
>    .section SECTION(.text), "ax", @progbits
>  ENTRY(STRNCPY)
> +# ifdef __ILP32__
> +  /* Clear the upper 32 bits.  */
> +  movl  %edx, %edx
> +# endif
>    /* Filter zero length strings and very long strings.  Zero
>       length strings just return, very long strings are handled by
>       just running rep stos{b|l} to zero set (which will almost

evex version will need the same treatment (assuming that works).

-- 
You are receiving this mail because:
You are on the CC list for the bug.