rs6000: Fix up __m64 typedef in mmintrin.h [PR97301]
authorJakub Jelinek <jakub@redhat.com>
Sat, 23 Jan 2021 08:41:58 +0000 (09:41 +0100)
committerJakub Jelinek <jakub@redhat.com>
Sat, 23 Jan 2021 08:41:58 +0000 (09:41 +0100)
The x86 __m64 type is defined as:
/* The Intel API is flexible enough that we must allow aliasing with other
   vector types, and their scalar components.  */
typedef int __m64 __attribute__ ((__vector_size__ (8), __may_alias__));
and so matches the comment above it in that reads and stores through
pointers to __m64 can alias anything.
But in the rs6000 headers that is the case only for __m128, but not __m64.

The following patch adds that attribute, which fixes the
FAIL: gcc.target/powerpc/sse-movhps-1.c execution test
FAIL: gcc.target/powerpc/sse-movlps-1.c execution test
regressions that appeared when Honza improved ipa-modref.

2021-01-23  Jakub Jelinek  <jakub@redhat.com>

PR testsuite/97301
* config/rs6000/mmintrin.h (__m64): Add __may_alias__ attribute.

gcc/config/rs6000/mmintrin.h

index 60d1a25d64b9c912e38aa19a38e5876436328583..0bd929c5afe9ca6ed3225f035bcd5321aeaad693 100644 (file)
@@ -58,7 +58,8 @@
 #include <altivec.h>
 /* The Intel API is flexible enough that we must allow aliasing with other
    vector types, and their scalar components.  */
-typedef __attribute__ ((__aligned__ (8))) unsigned long long __m64;
+typedef __attribute__ ((__aligned__ (8),
+                       __may_alias__)) unsigned long long __m64;
 
 typedef __attribute__ ((__aligned__ (8)))
 union