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