On Wed, Jul 19, 2017 at 12:33:03PM -0500, Segher Boessenkool wrote:
> 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.

Yes, I'm only changing the one test that is in the GCC 7/6 branches.  The other
tests that use the __builtin_cpu_supports built-in (the bmi* tests) will
continue to use the target supports method.

-- 
Michael Meissner, IBM
IBM, M/S 2506R, 550 King Street, Littleton, MA 01460-6245, USA
email: meiss...@linux.vnet.ibm.com, phone: +1 (978) 899-4797

Reply via email to