On Tue, Jan 26, 2021 at 06:29:47PM +0100, Jakub Jelinek wrote: > On Sat, Jan 23, 2021 at 03:10:10PM -0600, Segher Boessenkool wrote: > > > The reason I chose the "no-strict-aliasing" attribute (and already > > > committed based on Richi's ack) was consistency with the i386 > > > testcase. I can change both. > > > > Yes please. Thanks! > > So like this? > Tested on x86_64-linux (-m32/-m64) and powerpc64le-linux, ok for trunk?
That looks like it should work fine. Thank you! The rs6000 part is okay for trunk. (The x86 part looks fine to me as well, fwiw). Segher > 2021-01-26 Jakub Jelinek <ja...@redhat.com> > > * gcc.target/powerpc/m128-check.h (CHECK_EXP): Remove > optimize ("no-strict-aliasing") attribute. > * gcc.target/powerpc/sse-andnps-1.c (TEST): Copy e into float[4] > array to avoid violating TBAA. > * gcc.target/powerpc/sse2-andpd-1.c (TEST): Copy e.d into double[2] > array to avoid violating TBAA. > * gcc.target/powerpc/sse-andps-1.c (TEST): Copy e.f into float[4] > array to avoid violating TBAA. > * gcc.target/powerpc/sse2-andnpd-1.c (TEST): Copy e into double[2] > array to avoid violating TBAA.