<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Tue, May 7, 2024 at 11:59 AM Sunil Pandey <<a href="mailto:skpgkp2@gmail.com">skpgkp2@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, May 7, 2024 at 11:27 AM Gabi Falk <<a href="mailto:gabifalk@gmx.com" target="_blank">gabifalk@gmx.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">This code expects the WCSCAT preprocessor macro to be predefined in case<br>
the evex implementation of the function should be defined with a name<br>
different from __wcsncat_evex.  However, when glibc is built for<br>
x86-64-v4 without multiarch support, sysdeps/x86_64/wcsncat.S defines<br>
WCSNCAT variable instead of WCSCAT to build it as wcsncat.  Rename the<br>
variable to WCSNCAT, as it is actually a better naming choice for the<br>
variable in this case.<br>
<br>
Reported-by: Kenton Groombridge<br>
Link: <a href="https://bugs.gentoo.org/921945" rel="noreferrer" target="_blank">https://bugs.gentoo.org/921945</a><br>
Fixes: 64b8b6516b ("x86: Add evex optimized functions for the wchar_t strcpy family")<br>
Signed-off-by: Gabi Falk <<a href="mailto:gabifalk@gmx.com" target="_blank">gabifalk@gmx.com</a>><br>
---<br>
 sysdeps/x86_64/multiarch/wcsncat-evex.S | 6 +++---<br>
 1 file changed, 3 insertions(+), 3 deletions(-)<br>
<br>
diff --git a/sysdeps/x86_64/multiarch/wcsncat-evex.S b/sysdeps/x86_64/multiarch/wcsncat-evex.S<br>
index 392215950a..10bfb0a531 100644<br>
--- a/sysdeps/x86_64/multiarch/wcsncat-evex.S<br>
+++ b/sysdeps/x86_64/multiarch/wcsncat-evex.S<br>
@@ -1,9 +1,9 @@<br>
-#ifndef WCSCAT<br>
-# define WCSCAT        __wcsncat_evex<br>
+#ifndef WCSNCAT<br>
+# define WCSNCAT       __wcsncat_evex<br>
 #endif<br>
<br>
 #define USE_AS_WCSCPY<br>
 #define USE_AS_STRCAT<br>
<br>
-#define STRNCAT        WCSCAT<br>
+#define STRNCAT        WCSNCAT<br>
 #include "strncat-evex.S"<br>
--<br>
gabi<br>
<br></blockquote><div><br></div><div>LGTM</div><div>Reviewed-by: Sunil K Pandey <<a href="mailto:skpgkp2@gmail.com" target="_blank">skpgkp2@gmail.com</a>></div><div><br></div><div>--Sunil </div></div></div></blockquote><div><br></div><div><p class="MsoNormal" style="margin:0in;font-size:11pt;font-family:Aptos,sans-serif">I would like to backport this patch to release branches.<br>
Any comments or objections?<br>
<br>
--Sunil</p></div><div> </div></div></div>