This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, rs6000] Undefine vector, bool, pixel in xmmintrin.h
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Bill Schmidt <wschmidt at linux dot ibm dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Segher Boessenkool <segher at kernel dot crashing dot org>
- Date: Wed, 4 Apr 2018 10:53:34 +0200
- Subject: Re: [PATCH, rs6000] Undefine vector, bool, pixel in xmmintrin.h
- References: <db4f94be-b2de-504c-b458-133a85248902@linux.ibm.com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Sun, Apr 01, 2018 at 08:24:32PM -0500, Bill Schmidt wrote:
> I also updated the gcc.target/powerpc/powerpc.exp file to allow C++
> tests to be placed in that directory (with a *.C suffix).
I think this is wrong.
Historically, we've been putting target C++ tests into g++.dg/*/*.C
with appropriate target guards, the gcc.target/ tests are compiled with the
gcc driver rather than g++, so in your case it just happens to work because
it is a compile time test only (and one that doesn't use any STL headers),
e.g. dg-do run test would fail miserably.
If we (for GCC9?) want to create a spot for target C++ tests, we should
just add g++.target/<cpu>/ directories and add all the needed infrastructure
for those.
Can you please revert the powerpc.exp change and move the test to
g++.dg/ext/ ? Thanks.
Jakub