This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH], Update cpu-builtin-1.c test on PowerPC
- From: Segher Boessenkool <segher at kernel dot crashing dot org>
- To: Michael Meissner <meissner at linux dot vnet dot ibm dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, David Edelsohn <dje dot gcc at gmail dot com>, Bill Schmidt <wschmidt at linux dot vnet dot ibm dot com>
- Date: Wed, 19 Jul 2017 12:33:03 -0500
- Subject: Re: [PATCH], Update cpu-builtin-1.c test on PowerPC
- Authentication-results: sourceware.org; auth=none
- References: <20170718202446.GA25162@ibm-tiger.the-meissners.org>
On Tue, Jul 18, 2017 at 04:24:47PM -0400, Michael Meissner wrote:
> This patch modifies the change I made on July 12th. It modifies the test for
> the __builtin_cpu_is and __builtin_cpu_supports built-in functions to use an
> #ifdef instead of target-requires for doing the tests. One motavation is to
> make the back port to GCC 6/7 easier, as I won't have to back port the change
> to add the target option ppc_cpu_supports_hw.
>
> I've checked the trunk with compilers built with a new GLIBC and without, and
> it passes both compilers. I also checked the back port to GCC 6/7 and both
> work fine as well.
>
> Can I check this patch into the trunk and backports to GCC 6 and 7?
You will only do this for the tests of __builtin_cpu_* itself? Okay
(for all branches) in that case.
Segher
> 2017-07-18 Michael Meissner <meissner@linux.vnet.ibm.com>
>
> PR target/81193
> * gcc.target/powerpc/cpu-builtin-1.c: Change test to use #ifdef
> __BUILTIN_CPU_SUPPORTS to see if the GLIBC is new enough that
> __builtin_cpu_is and __builtin_cpu_supports are supported.