* [PATCH] x86-64: Define __memcmpeq in ld.so
@ 2022-02-14 2:46 H.J. Lu
2022-02-14 9:17 ` Noah Goldstein
0 siblings, 1 reply; 5+ messages in thread
From: H.J. Lu @ 2022-02-14 2:46 UTC (permalink / raw)
To: libc-alpha
Define __memcmpeq in ld.so so that compiler can generate __memcmpeq call
when compiling for ld.so.
---
sysdeps/x86_64/multiarch/memcmpeq-sse2.S | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/sysdeps/x86_64/multiarch/memcmpeq-sse2.S b/sysdeps/x86_64/multiarch/memcmpeq-sse2.S
index b80a29d4b0..de7f5a7525 100644
--- a/sysdeps/x86_64/multiarch/memcmpeq-sse2.S
+++ b/sysdeps/x86_64/multiarch/memcmpeq-sse2.S
@@ -16,8 +16,10 @@
License along with the GNU C Library; if not, see
<https://www.gnu.org/licenses/>. */
-#ifndef memcmp
+#if IS_IN (libc)
# define memcmp __memcmpeq_sse2
+#else
+# define memcmp __memcmpeq
#endif
#define USE_AS_MEMCMPEQ 1
#include "memcmp-sse2.S"
--
2.34.1
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] x86-64: Define __memcmpeq in ld.so
2022-02-14 2:46 [PATCH] x86-64: Define __memcmpeq in ld.so H.J. Lu
@ 2022-02-14 9:17 ` Noah Goldstein
2022-02-14 13:49 ` H.J. Lu
0 siblings, 1 reply; 5+ messages in thread
From: Noah Goldstein @ 2022-02-14 9:17 UTC (permalink / raw)
To: H.J. Lu; +Cc: GNU C Library
On Sun, Feb 13, 2022 at 8:46 PM H.J. Lu <hjl.tools@gmail.com> wrote:
>
> Define __memcmpeq in ld.so so that compiler can generate __memcmpeq call
> when compiling for ld.so.
> ---
> sysdeps/x86_64/multiarch/memcmpeq-sse2.S | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/sysdeps/x86_64/multiarch/memcmpeq-sse2.S b/sysdeps/x86_64/multiarch/memcmpeq-sse2.S
> index b80a29d4b0..de7f5a7525 100644
> --- a/sysdeps/x86_64/multiarch/memcmpeq-sse2.S
> +++ b/sysdeps/x86_64/multiarch/memcmpeq-sse2.S
> @@ -16,8 +16,10 @@
> License along with the GNU C Library; if not, see
> <https://www.gnu.org/licenses/>. */
>
> -#ifndef memcmp
> +#if IS_IN (libc)
> # define memcmp __memcmpeq_sse2
> +#else
> +# define memcmp __memcmpeq
> #endif
> #define USE_AS_MEMCMPEQ 1
> #include "memcmp-sse2.S"
How can I test this patch?
> --
> 2.34.1
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] x86-64: Define __memcmpeq in ld.so
2022-02-14 9:17 ` Noah Goldstein
@ 2022-02-14 13:49 ` H.J. Lu
2022-02-15 1:20 ` Noah Goldstein
0 siblings, 1 reply; 5+ messages in thread
From: H.J. Lu @ 2022-02-14 13:49 UTC (permalink / raw)
To: Noah Goldstein; +Cc: GNU C Library
On Mon, Feb 14, 2022 at 1:18 AM Noah Goldstein <goldstein.w.n@gmail.com> wrote:
>
> On Sun, Feb 13, 2022 at 8:46 PM H.J. Lu <hjl.tools@gmail.com> wrote:
> >
> > Define __memcmpeq in ld.so so that compiler can generate __memcmpeq call
> > when compiling for ld.so.
> > ---
> > sysdeps/x86_64/multiarch/memcmpeq-sse2.S | 4 +++-
> > 1 file changed, 3 insertions(+), 1 deletion(-)
> >
> > diff --git a/sysdeps/x86_64/multiarch/memcmpeq-sse2.S b/sysdeps/x86_64/multiarch/memcmpeq-sse2.S
> > index b80a29d4b0..de7f5a7525 100644
> > --- a/sysdeps/x86_64/multiarch/memcmpeq-sse2.S
> > +++ b/sysdeps/x86_64/multiarch/memcmpeq-sse2.S
> > @@ -16,8 +16,10 @@
> > License along with the GNU C Library; if not, see
> > <https://www.gnu.org/licenses/>. */
> >
> > -#ifndef memcmp
> > +#if IS_IN (libc)
> > # define memcmp __memcmpeq_sse2
> > +#else
> > +# define memcmp __memcmpeq
> > #endif
> > #define USE_AS_MEMCMPEQ 1
> > #include "memcmp-sse2.S"
>
> How can I test this patch?
Please build this GCC branch:
https://gitlab.com/x86-gcc/gcc/-/commits/users/hjl/memcmpeq/master
and use it to build glibc with
CC="gcc -fextra-libc-function=memcmpeq"
It will call __memcmpeq when possible.
--
H.J.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] x86-64: Define __memcmpeq in ld.so
2022-02-14 13:49 ` H.J. Lu
@ 2022-02-15 1:20 ` Noah Goldstein
2022-05-12 19:24 ` Sunil Pandey
0 siblings, 1 reply; 5+ messages in thread
From: Noah Goldstein @ 2022-02-15 1:20 UTC (permalink / raw)
To: H.J. Lu; +Cc: GNU C Library
On Mon, Feb 14, 2022 at 7:50 AM H.J. Lu <hjl.tools@gmail.com> wrote:
>
> On Mon, Feb 14, 2022 at 1:18 AM Noah Goldstein <goldstein.w.n@gmail.com> wrote:
> >
> > On Sun, Feb 13, 2022 at 8:46 PM H.J. Lu <hjl.tools@gmail.com> wrote:
> > >
> > > Define __memcmpeq in ld.so so that compiler can generate __memcmpeq call
> > > when compiling for ld.so.
> > > ---
> > > sysdeps/x86_64/multiarch/memcmpeq-sse2.S | 4 +++-
> > > 1 file changed, 3 insertions(+), 1 deletion(-)
> > >
> > > diff --git a/sysdeps/x86_64/multiarch/memcmpeq-sse2.S b/sysdeps/x86_64/multiarch/memcmpeq-sse2.S
> > > index b80a29d4b0..de7f5a7525 100644
> > > --- a/sysdeps/x86_64/multiarch/memcmpeq-sse2.S
> > > +++ b/sysdeps/x86_64/multiarch/memcmpeq-sse2.S
> > > @@ -16,8 +16,10 @@
> > > License along with the GNU C Library; if not, see
> > > <https://www.gnu.org/licenses/>. */
> > >
> > > -#ifndef memcmp
> > > +#if IS_IN (libc)
> > > # define memcmp __memcmpeq_sse2
> > > +#else
> > > +# define memcmp __memcmpeq
> > > #endif
> > > #define USE_AS_MEMCMPEQ 1
> > > #include "memcmp-sse2.S"
> >
> > How can I test this patch?
>
> Please build this GCC branch:
>
> https://gitlab.com/x86-gcc/gcc/-/commits/users/hjl/memcmpeq/master
>
> and use it to build glibc with
>
> CC="gcc -fextra-libc-function=memcmpeq"
>
> It will call __memcmpeq when possible.
LGTM.
>
> --
> H.J.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] x86-64: Define __memcmpeq in ld.so
2022-02-15 1:20 ` Noah Goldstein
@ 2022-05-12 19:24 ` Sunil Pandey
0 siblings, 0 replies; 5+ messages in thread
From: Sunil Pandey @ 2022-05-12 19:24 UTC (permalink / raw)
To: Noah Goldstein, Libc-stable Mailing List; +Cc: H.J. Lu, GNU C Library
On Mon, Feb 14, 2022 at 5:20 PM Noah Goldstein via Libc-alpha
<libc-alpha@sourceware.org> wrote:
>
> On Mon, Feb 14, 2022 at 7:50 AM H.J. Lu <hjl.tools@gmail.com> wrote:
> >
> > On Mon, Feb 14, 2022 at 1:18 AM Noah Goldstein <goldstein.w.n@gmail.com> wrote:
> > >
> > > On Sun, Feb 13, 2022 at 8:46 PM H.J. Lu <hjl.tools@gmail.com> wrote:
> > > >
> > > > Define __memcmpeq in ld.so so that compiler can generate __memcmpeq call
> > > > when compiling for ld.so.
> > > > ---
> > > > sysdeps/x86_64/multiarch/memcmpeq-sse2.S | 4 +++-
> > > > 1 file changed, 3 insertions(+), 1 deletion(-)
> > > >
> > > > diff --git a/sysdeps/x86_64/multiarch/memcmpeq-sse2.S b/sysdeps/x86_64/multiarch/memcmpeq-sse2.S
> > > > index b80a29d4b0..de7f5a7525 100644
> > > > --- a/sysdeps/x86_64/multiarch/memcmpeq-sse2.S
> > > > +++ b/sysdeps/x86_64/multiarch/memcmpeq-sse2.S
> > > > @@ -16,8 +16,10 @@
> > > > License along with the GNU C Library; if not, see
> > > > <https://www.gnu.org/licenses/>. */
> > > >
> > > > -#ifndef memcmp
> > > > +#if IS_IN (libc)
> > > > # define memcmp __memcmpeq_sse2
> > > > +#else
> > > > +# define memcmp __memcmpeq
> > > > #endif
> > > > #define USE_AS_MEMCMPEQ 1
> > > > #include "memcmp-sse2.S"
> > >
> > > How can I test this patch?
> >
> > Please build this GCC branch:
> >
> > https://gitlab.com/x86-gcc/gcc/-/commits/users/hjl/memcmpeq/master
> >
> > and use it to build glibc with
> >
> > CC="gcc -fextra-libc-function=memcmpeq"
> >
> > It will call __memcmpeq when possible.
>
> LGTM.
> >
> > --
> > H.J.
I would like to backport this patch to release branches.
Any comments or objections?
--Sunil
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2022-05-12 19:25 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-02-14 2:46 [PATCH] x86-64: Define __memcmpeq in ld.so H.J. Lu
2022-02-14 9:17 ` Noah Goldstein
2022-02-14 13:49 ` H.J. Lu
2022-02-15 1:20 ` Noah Goldstein
2022-05-12 19:24 ` Sunil Pandey
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).