Re: [PATCH] rs6000: Fix up #include or [PR104239]
On Thu, Jan 27, 2022 at 02:55:08PM -0600, Paul A. Clarke wrote: > Should we add similar compile-only tests for all of the standalone include > files? The only reason it wasn't detected earlier is that no one actually uses this (and tests the development compiler). That is a much bigger problem, and nothing that a few scattershot tests will help fix :-( Segher
Re: [PATCH] rs6000: Fix up #include or [PR104239]
On Wed, Jan 26, 2022 at 03:50:35PM -0500, David Edelsohn via Gcc-patches wrote: > On Wed, Jan 26, 2022 at 3:45 PM Jakub Jelinek wrote: > > r12-4717-g7d37abedf58d66 added immintrin.h and x86gprintrin.h headers > > to rs6000, these headers are on x86 standalone headers that various > > programs include directly rather than including them through > > . > > Unfortunately, for that change the bmiintrin.h and bmi2intrin.h > > headers haven't been adjusted, so the effect is that if one includes them > > (without including also x86intrin.h first) #error will trigger. > > Furthermore, when including such headers conditionally as some real-world > > packages do, this means a regression. > > > > The following patch fixes it and matches what the x86 bmi{,2}intrin.h > > headers do. > > Okay. > > Thanks for catching this. Indeed, thanks. And thanks for reviewing, David. Should we add similar compile-only tests for all of the standalone include files? PC
Re: [PATCH] rs6000: Fix up #include or [PR104239]
On Wed, Jan 26, 2022 at 3:45 PM Jakub Jelinek wrote: > > Hi! > > r12-4717-g7d37abedf58d66 added immintrin.h and x86gprintrin.h headers > to rs6000, these headers are on x86 standalone headers that various > programs include directly rather than including them through > . > Unfortunately, for that change the bmiintrin.h and bmi2intrin.h > headers haven't been adjusted, so the effect is that if one includes them > (without including also x86intrin.h first) #error will trigger. > Furthermore, when including such headers conditionally as some real-world > packages do, this means a regression. > > The following patch fixes it and matches what the x86 bmi{,2}intrin.h > headers do. > > Bootstrapped/regtested on powerpc64le-linux, ok for trunk? Okay. Thanks for catching this. - David > > 2022-01-26 Jakub Jelinek > > PR target/104239 > * config/rs6000/bmiintrin.h: Test _X86GPRINTRIN_H_INCLUDED instead of > _X86INTRIN_H_INCLUDED and adjust #error wording. > * config/rs6000/bmi2intrin.h: Likewise. > > * gcc.target/powerpc/pr104239-1.c: New test. > * gcc.target/powerpc/pr104239-2.c: New test. > > --- gcc/config/rs6000/bmiintrin.h.jj2022-01-11 23:11:21.936296534 +0100 > +++ gcc/config/rs6000/bmiintrin.h 2022-01-26 13:35:08.705945170 +0100 > @@ -29,8 +29,8 @@ > standard C or GNU C extensions, which are more portable and better > optimized across multiple targets. */ > > -#if !defined _X86INTRIN_H_INCLUDED > -# error "Never use directly; include instead." > +#if !defined _X86GPRINTRIN_H_INCLUDED > +# error "Never use directly; include instead." > #endif > > #ifndef _BMIINTRIN_H_INCLUDED > --- gcc/config/rs6000/bmi2intrin.h.jj 2022-01-11 23:11:21.936296534 +0100 > +++ gcc/config/rs6000/bmi2intrin.h 2022-01-26 13:34:53.373162122 +0100 > @@ -29,8 +29,8 @@ > standard C or GNU C extensions, which are more portable and better > optimized across multiple targets. */ > > -#if !defined _X86INTRIN_H_INCLUDED > -# error "Never use directly; include instead." > +#if !defined _X86GPRINTRIN_H_INCLUDED > +# error "Never use directly; include > instead." > #endif > > #ifndef _BMI2INTRIN_H_INCLUDED > --- gcc/testsuite/gcc.target/powerpc/pr104239-1.c.jj2022-01-26 > 13:42:34.103643030 +0100 > +++ gcc/testsuite/gcc.target/powerpc/pr104239-1.c 2022-01-26 > 13:42:23.101798701 +0100 > @@ -0,0 +1,9 @@ > +/* PR target/104239 */ > +/* { dg-do compile } */ > +/* { dg-options "-DNO_WARN_X86_INTRINSICS" } */ > + > +#if __has_include() > +#include > +#endif > + > +int i; > --- gcc/testsuite/gcc.target/powerpc/pr104239-2.c.jj2022-01-26 > 13:42:42.279527345 +0100 > +++ gcc/testsuite/gcc.target/powerpc/pr104239-2.c 2022-01-26 > 13:42:23.101798701 +0100 > @@ -0,0 +1,9 @@ > +/* PR target/104239 */ > +/* { dg-do compile } */ > +/* { dg-options "-DNO_WARN_X86_INTRINSICS" } */ > + > +#if __has_include() > +#include > +#endif > + > +int i; > > Jakub >
[PATCH] rs6000: Fix up #include or [PR104239]
Hi! r12-4717-g7d37abedf58d66 added immintrin.h and x86gprintrin.h headers to rs6000, these headers are on x86 standalone headers that various programs include directly rather than including them through . Unfortunately, for that change the bmiintrin.h and bmi2intrin.h headers haven't been adjusted, so the effect is that if one includes them (without including also x86intrin.h first) #error will trigger. Furthermore, when including such headers conditionally as some real-world packages do, this means a regression. The following patch fixes it and matches what the x86 bmi{,2}intrin.h headers do. Bootstrapped/regtested on powerpc64le-linux, ok for trunk? 2022-01-26 Jakub Jelinek PR target/104239 * config/rs6000/bmiintrin.h: Test _X86GPRINTRIN_H_INCLUDED instead of _X86INTRIN_H_INCLUDED and adjust #error wording. * config/rs6000/bmi2intrin.h: Likewise. * gcc.target/powerpc/pr104239-1.c: New test. * gcc.target/powerpc/pr104239-2.c: New test. --- gcc/config/rs6000/bmiintrin.h.jj2022-01-11 23:11:21.936296534 +0100 +++ gcc/config/rs6000/bmiintrin.h 2022-01-26 13:35:08.705945170 +0100 @@ -29,8 +29,8 @@ standard C or GNU C extensions, which are more portable and better optimized across multiple targets. */ -#if !defined _X86INTRIN_H_INCLUDED -# error "Never use directly; include instead." +#if !defined _X86GPRINTRIN_H_INCLUDED +# error "Never use directly; include instead." #endif #ifndef _BMIINTRIN_H_INCLUDED --- gcc/config/rs6000/bmi2intrin.h.jj 2022-01-11 23:11:21.936296534 +0100 +++ gcc/config/rs6000/bmi2intrin.h 2022-01-26 13:34:53.373162122 +0100 @@ -29,8 +29,8 @@ standard C or GNU C extensions, which are more portable and better optimized across multiple targets. */ -#if !defined _X86INTRIN_H_INCLUDED -# error "Never use directly; include instead." +#if !defined _X86GPRINTRIN_H_INCLUDED +# error "Never use directly; include instead." #endif #ifndef _BMI2INTRIN_H_INCLUDED --- gcc/testsuite/gcc.target/powerpc/pr104239-1.c.jj2022-01-26 13:42:34.103643030 +0100 +++ gcc/testsuite/gcc.target/powerpc/pr104239-1.c 2022-01-26 13:42:23.101798701 +0100 @@ -0,0 +1,9 @@ +/* PR target/104239 */ +/* { dg-do compile } */ +/* { dg-options "-DNO_WARN_X86_INTRINSICS" } */ + +#if __has_include() +#include +#endif + +int i; --- gcc/testsuite/gcc.target/powerpc/pr104239-2.c.jj2022-01-26 13:42:42.279527345 +0100 +++ gcc/testsuite/gcc.target/powerpc/pr104239-2.c 2022-01-26 13:42:23.101798701 +0100 @@ -0,0 +1,9 @@ +/* PR target/104239 */ +/* { dg-do compile } */ +/* { dg-options "-DNO_WARN_X86_INTRINSICS" } */ + +#if __has_include() +#include +#endif + +int i; Jakub