[PATCH] D20617: [X86][SSE] _mm_store1_ps/_mm_store1_pd should require an aligned pointer

Simon Pilgrim via cfe-commits cfe-commits at lists.llvm.org
Fri May 27 10:05:34 PDT 2016


RKSimon added inline comments.

================
Comment at: lib/Headers/emmintrin.h:598
@@ -594,3 +597,3 @@
 static __inline__ void __DEFAULT_FN_ATTRS
-_mm_store_pd(double *__dp, __m128d __a)
+_mm_store_pd1(double *__dp, __m128d __a)
 {
----------------
majnemer wrote:
> You could use `__attribute__((align_value(16)))` no?
Technically yes but AFAICT there are no other users of this approach in the headers - is it something that we should be encouraging do you think?

Craig - I think you wrote in a commit about dropping the unaligned intrinsics, is that how you'd do it? 


Repository:
  rL LLVM

http://reviews.llvm.org/D20617





More information about the cfe-commits mailing list