Mesa (master): i965: Use standard SSE intrinsics instead of gcc built-ins.

Matt Turner mattst88 at kemper.freedesktop.org
Tue Jun 17 17:18:54 UTC 2014


Module: Mesa
Branch: master
Commit: 83649587c6a10a99d00a01eb72a6563a61aaf8a7
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=83649587c6a10a99d00a01eb72a6563a61aaf8a7

Author: Matt Turner <mattst88 at gmail.com>
Date:   Sat Jun 14 22:31:33 2014 -0700

i965: Use standard SSE intrinsics instead of gcc built-ins.

Let's this file compile with clang.

Reviewed-by: Frank Henigman <fjhenigman at google.com>
Reviewed-by: Tapani Pälli <tapani.palli at intel.com>
Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

---

 src/mesa/drivers/dri/i965/intel_tex_subimage.c |   12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/intel_tex_subimage.c b/src/mesa/drivers/dri/i965/intel_tex_subimage.c
index 5d79750..9cfff7a 100644
--- a/src/mesa/drivers/dri/i965/intel_tex_subimage.c
+++ b/src/mesa/drivers/dri/i965/intel_tex_subimage.c
@@ -42,6 +42,10 @@
 #include "intel_mipmap_tree.h"
 #include "intel_blit.h"
 
+#ifdef __SSSE3__
+#include <tmmintrin.h>
+#endif
+
 #define FILE_DEBUG_FLAG DEBUG_TEXTURE
 
 #define ALIGN_DOWN(a, b) ROUND_DOWN_TO(a, b)
@@ -174,13 +178,11 @@ err:
 static const uint8_t rgba8_permutation[16] =
    { 2,1,0,3, 6,5,4,7, 10,9,8,11, 14,13,12,15 };
 
-typedef char v16 __attribute__((vector_size(16)));
-
 /* NOTE: dst must be 16 byte aligned */
 #define rgba8_copy_16(dst, src)                     \
-   *(v16*)(dst) = __builtin_ia32_pshufb128(         \
-       (v16) __builtin_ia32_loadups((float*)(src)), \
-      *(v16*) rgba8_permutation                     \
+   *(__m128i *)(dst) = _mm_shuffle_epi8(            \
+      (__m128i) _mm_loadu_ps((float *)(src)),       \
+      *(__m128i *) rgba8_permutation                \
    )
 #endif
 




More information about the mesa-commit mailing list