x86-64: Don't mark symbols as hidden in strcmp-XXX.S
Checks
Context |
Check |
Description |
dj/TryBot-apply_patch |
success
|
Patch applied to master at the time it was sent
|
dj/TryBot-32bit |
success
|
Build for i686
|
Commit Message
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(-)
Comments
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?
> # 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
>
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
> >
>
>
> --
> H.J.
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?
> >
> > --
> > H.J.
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.
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.
Let's leave it asis for now. We only need to remove .hidden.
> > > # 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
> > >
> >
> >
> > --
> > H.J.
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?
>
>
> --
> H.J.
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.
On Thu, Jul 7, 2022 at 4:58 PM H.J. Lu <hjl.tools@gmail.com> wrote:
>
> 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.
Okay.
>
> --
> H.J.
On Thu, Jul 7, 2022 at 5:00 PM Noah Goldstein <goldstein.w.n@gmail.com> wrote:
>
> On Thu, Jul 7, 2022 at 4:58 PM H.J. Lu <hjl.tools@gmail.com> wrote:
> >
> > 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.
>
> Okay.
I am checking it in.
Thanks.
On Thu, Jul 7, 2022 at 5:06 PM H.J. Lu <hjl.tools@gmail.com> wrote:
>
> On Thu, Jul 7, 2022 at 5:00 PM Noah Goldstein <goldstein.w.n@gmail.com> wrote:
> >
> > On Thu, Jul 7, 2022 at 4:58 PM H.J. Lu <hjl.tools@gmail.com> wrote:
> > >
> > > 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.
> >
> > Okay.
>
> I am checking it in.
>
Yup,
LGTM.
> Thanks.
>
> --
> H.J.
@@ -184,7 +184,6 @@
.align 16
.type STRCMP, @function
.globl STRCMP
- .hidden STRCMP
# ifndef GLABEL
# define GLABEL(...) __VA_ARGS__
@@ -214,7 +214,6 @@
.align 16
.type STRCMP, @function
.globl STRCMP
- .hidden STRCMP
# ifdef USE_AS_STRCASECMP_L
ENTRY (STRCASECMP)
@@ -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