[PATCH] x86-64: Don't mark symbols as hidden in strcmp-XXX.S
H.J. Lu
hjl.tools@gmail.com
Thu Jul 7 23:57:42 GMT 2022
On Thu, Jul 7, 2022 at 4:54 PM Noah Goldstein <goldstein.w.n@gmail.com> wrote:
>
> On Thu, Jul 7, 2022 at 4:52 PM H.J. Lu <hjl.tools@gmail.com> wrote:
> >
> > On Thu, Jul 7, 2022 at 4:47 PM Noah Goldstein <goldstein.w.n@gmail.com> wrote:
> > >
> > > On Thu, Jul 7, 2022 at 4:45 PM Noah Goldstein <goldstein.w.n@gmail.com> wrote:
> > > >
> > > > On Thu, Jul 7, 2022 at 4:43 PM H.J. Lu <hjl.tools@gmail.com> wrote:
> > > > >
> > > > > On Thu, Jul 7, 2022 at 4:39 PM H.J. Lu <hjl.tools@gmail.com> wrote:
> > > > > >
> > > > > > Don't mark symbols as hidden in strcmp-avx2.S, strcmp-evex.S and
> > > > > > strcmp-sse42.S since they are marked as hidden in the IFUNC selectors.
> > > > > > ---
> > > > > > sysdeps/x86_64/multiarch/strcmp-avx2.S | 1 -
> > > > > > sysdeps/x86_64/multiarch/strcmp-evex.S | 1 -
> > > > > > sysdeps/x86_64/multiarch/strcmp-sse42.S | 1 -
> > > > > > 3 files changed, 3 deletions(-)
> > > > > >
> > > > > > diff --git a/sysdeps/x86_64/multiarch/strcmp-avx2.S b/sysdeps/x86_64/multiarch/strcmp-avx2.S
> > > > > > index 8da09bd86d..516cde1145 100644
> > > > > > --- a/sysdeps/x86_64/multiarch/strcmp-avx2.S
> > > > > > +++ b/sysdeps/x86_64/multiarch/strcmp-avx2.S
> > > > > > @@ -184,7 +184,6 @@
> > > > > > .align 16
> > > > > > .type STRCMP, @function
> > > > > > .globl STRCMP
> > > > > > - .hidden STRCMP
> > > > >
> > > > > Hi Noah,
> > > > >
> > > > > Why aren't ENTRY/END used on STRCMP?
> > > > >
> > > >
> > > > Not sure, I copied sse42 when doing avx2/evex.
> > > >
> > > > > > # ifndef GLABEL
> > > > > > # define GLABEL(...) __VA_ARGS__
> > > > > > diff --git a/sysdeps/x86_64/multiarch/strcmp-evex.S b/sysdeps/x86_64/multiarch/strcmp-evex.S
> > > > > > index 2a5b3ce037..e97d51bb26 100644
> > > > > > --- a/sysdeps/x86_64/multiarch/strcmp-evex.S
> > > > > > +++ b/sysdeps/x86_64/multiarch/strcmp-evex.S
> > > > > > @@ -214,7 +214,6 @@
> > > > > > .align 16
> > > > > > .type STRCMP, @function
> > > > > > .globl STRCMP
> > > > > > - .hidden STRCMP
> > > > > >
> > > > > > # ifdef USE_AS_STRCASECMP_L
> > > > > > ENTRY (STRCASECMP)
> > > > > > diff --git a/sysdeps/x86_64/multiarch/strcmp-sse42.S b/sysdeps/x86_64/multiarch/strcmp-sse42.S
> > > > > > index a9178ad25c..60313c647a 100644
> > > > > > --- a/sysdeps/x86_64/multiarch/strcmp-sse42.S
> > > > > > +++ b/sysdeps/x86_64/multiarch/strcmp-sse42.S
> > > > > > @@ -77,7 +77,6 @@
> > > > > > .align 16
> > > > > > .type STRCMP_SSE42, @function
> > > > > > .globl STRCMP_SSE42
> > > > > > - .hidden STRCMP_SSE42
> > > > > > #ifdef USE_AS_STRCASECMP_L
> > > > > > ENTRY (GLABEL(__strcasecmp))
> > > > > > movq __libc_tsd_LOCALE@gottpoff(%rip),%rax
> > > > > > --
> > > > > > 2.36.1
> > > > > >
> > > > >
> > >
> > > Do you need to do strcmp-sse2.S?
> >
> > There is no strcmp-sse2.S on master branch.
>
> Can we rebase this ontop of my patches?
> Between the "move stuff" and isa raising?
It should be fixed on master branch first. We don't need
# ifdef USE_MULTIARCH
.hidden STRCMP
# endif
It isn't necessary.
--
H.J.
More information about the Libc-alpha
mailing list