This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH rs6000] Fix up BMI/BMI2 intrinsic DG tests
- From: Segher Boessenkool <segher at kernel dot crashing dot org>
- To: Steven Munroe <munroesj at linux dot vnet dot ibm dot com>
- Cc: gcc-patches <gcc-patches at gcc dot gnu dot org>, David Edelsohn <dje dot gcc at gmail dot com>
- Date: Tue, 18 Jul 2017 16:54:30 -0500
- Subject: Re: [PATCH rs6000] Fix up BMI/BMI2 intrinsic DG tests
- Authentication-results: sourceware.org; auth=none
- References: <1500316101.22326.11.camel@oc7878010663>
Hi!
On Mon, Jul 17, 2017 at 01:28:20PM -0500, Steven Munroe wrote:
> After a resent GCC change the previously submitted BMI/BMI2 intrinsic
> test started to fail with the following warning/error.
>
> ppc_cpu_supports_hw_available122373.c: In function 'main':
> ppc_cpu_supports_hw_available122373.c:9:10: warning:
> __builtin_cpu_supports need
> s GLIBC (2.23 and newer) that exports hardware capability bits
>
> The does not occur on systems with the newer (2.23) GLIBC but is common
> on older (stable) distos.
>
> As this is coming from the bmi-check.h and bmi2-check.h includes (and
> not the tests directly) it seems simpler to simply skip the test unless
> __BUILTIN_CPU_SUPPORTS__ is defined.
So this will skip on most current systems; is there no reasonable
way around that?
Okay otherwise. One typo thing:
> 2017-07-17 Steven Munroe <munroesj@gcc.gnu.org>
>
> *gcc.target/powerpc/bmi-check.h (main): Skip unless
> __BUILTIN_CPU_SUPPORTS__ defined.
> *gcc.target/powerpc/bmi2-check.h (main): Skip unless
> __BUILTIN_CPU_SUPPORTS__ defined.
There should be a space after the asterisks.
Segher