[PATCH] elf/Makefile: Run IFUNC tests if binutils supports IFUNC

H.J. Lu hjl.tools@gmail.com
Fri Feb 22 14:37:00 GMT 2019


On Fri, Feb 22, 2019 at 6:35 AM Tulio Magno Quites Machado Filho
<tuliom@ascii.art.br> wrote:
>
> "H.J. Lu" <hjl.tools@gmail.com> writes:
>
> > We should run IFUNC tests with --disable-multi-arch if the toolchain
> > supports IFUNCs.  For correctness, --disable-multi-arch must not
> > remove IFUNC support from the loader.
> >
> > Tested on x86-64, x32 and i686 with and without --disable-multi-arch.
> >
> >       * configure.ac (have-ifunc): New LIBC_CONFIG_VAR.
> >       * configure: Regenerated.
> >       * elf/Makefile: Run IFUNC tests if binutils supports IFUNC.
>
> We should also change this powerpc test:
>
> diff --git a/sysdeps/powerpc/Makefile b/sysdeps/powerpc/Makefile
> index bac5a3a73c..5c6ea0a95e 100644
> --- a/sysdeps/powerpc/Makefile
> +++ b/sysdeps/powerpc/Makefile
> @@ -14,7 +14,7 @@ mod-tlsopt-powerpc.so-no-z-defs = yes
>  tests += tst-tlsopt-powerpc
>  $(objpfx)tst-tlsopt-powerpc: $(objpfx)mod-tlsopt-powerpc.so
>
> -ifneq (no,$(multi-arch))
> +ifeq (yes,$(have-ifunc))
>  tests-static += tst-tlsifunc-static
>  tests-internal += tst-tlsifunc-static
>  ifeq (yes,$(build-shared))
>
> Reviewed-by: Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com>
>

If all supported powerpc binutils have IFUNC, you can drop this test
like x86.


-- 
H.J.



More information about the Libc-alpha mailing list