This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] powerpc: Fix build failures with current GCC


<gabriel@inconstante.eti.br> writes:

> 	* sysdeps/powerpc/Makefile (CFLAGS-tst-set_ppr.c): New variable.

This entry is missing this:
[$(subdir) == misc]

> diff --git a/sysdeps/powerpc/Makefile b/sysdeps/powerpc/Makefile
> index 05675bc8ae..517a7297ec 100644
> --- a/sysdeps/powerpc/Makefile
> +++ b/sysdeps/powerpc/Makefile
> @@ -45,6 +45,7 @@ ifeq ($(subdir),misc)
>  sysdep_headers += sys/platform/ppc.h
>  tests += test-gettimebase
>  tests += tst-set_ppr
> +CFLAGS-tst-set_ppr.c += -Wa,-many

We need a comment explaining why this is needed, stating the test is expected
to run and exit with EXIT_UNSUPPORTED on processors that do not implement
the Power ISA 2.06 or greater.
But the test makes usage of instructions from Power ISA 2.06 and 2.07.

LGTM with these changes.

Reviewed-by: Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com>

-- 
Tulio Magno


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]