]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
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>
Fri, 29 Jan 2021 16:50:35 +0000 (17:50 +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 b735cbf3945de29cc2558645aee7d0b5b125ef22..497fb1528b482b3a9dfa122dd4709e42095d0625 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