[clang][Headers] _mm_stream_load_si128 should accept const arguments.

[ab]

Summary

Patch originally by Wolfgang Pieb.

[probinson]

Is line 157 now casting away const? Maybe it doesn't matter here...
(I can't seem to get phab to put my comment on the appropriate source line.)

[craig.topper]

I believe we should also fix the version in avxintrin.h _mm256_stream_load_si256(m256i *V)

[ab]

Stumbled upon this while looking for the intel guide! Fixed in r249213.

lib/Headers/smmintrin.h

 
 #define _mm_dp_pd(X, Y, M) __extension__ ({\
   __m128d __X = (X); \
   __m128d __Y = (Y); \
   (__m128d) __builtin_ia32_dppd((__v2df)__X, (__v2df)__Y, (M)); })
 
 /* SSE4 Streaming Load Hint Instruction.  */
 static __inline__  __m128i __attribute__((__always_inline__, __nodebug__))
-_mm_stream_load_si128 (__m128i *__V)
+_mm_stream_load_si128 (const __m128i *__V)
 {
   return (__m128i) __builtin_ia32_movntdqa ((__v2di *) __V);
 }
 
 /* SSE4 Packed Integer Min/Max Instructions.  */
 static __inline__  __m128i __attribute__((__always_inline__, __nodebug__))
 _mm_min_epi8 (__m128i __V1, __m128i __V2)
 {

test/CodeGen/builtins-x86-sse42.c

+// RUN: %clang_cc1 -triple x86_64-unknown-unknown -Werror -target-feature +sse4.2 %s -S -o - | FileCheck %s
+
+#define __MM_MALLOC_H
+
+#include <x86intrin.h>
+
+__m128i stream_load(__m128i *p)
+{
+  // CHECK-LABEL: stream_load
+  // CHECK: movntdqa
+  return _mm_stream_load_si128(p);
+}
+
+__m128i stream_load_c(const __m128i *p)
+{
+  // CHECK-LABEL: stream_load_c
+  // CHECK: movntdqa
+  return _mm_stream_load_si128(p);
+}