On Fri, Aug 18, 2017 at 04:49:44PM -0500, Steven Munroe wrote: > On Thu, 2017-08-17 at 00:47 -0500, Segher Boessenkool wrote: > > On Wed, Aug 16, 2017 at 03:50:55PM -0500, Steven Munroe wrote: > > > This it part 3/3 for contributing PPC64LE support for X86 SSE > > > instrisics. This patch includes testsuite/gcc.target tests for the > > > intrinsics included by xmmintrin.h. > > > > > +#define CHECK_EXP(UINON_TYPE, VALUE_TYPE, FMT) \ > > > > Should that be UNION_TYPE? > > It is spelled 'UINON_TYPE' in > ./gcc/testsuite/gcc.target/i386/m128-check.h which the source for the > powerpc version. > > There is no obvious reason why it could not be spelled UNION_TYPE. > Unless there is some symbol collision further up the SSE/AVX stack. > > Bingo: > > avx512f-helper.h:#define UNION_TYPE(SIZE, NAME) EVAL(union, SIZE, NAME) > > I propose not to change this.
Heh. Okay :-) Segher
