[Mesa-dev] [PATCH] swr/rastr: Don't assume non-WIN is always 64 bit, instead use the x86_64 define

Gert Wollny gw.fossdev at gmail.com
Sat May 26 21:20:46 UTC 2018

One must not assume that compiling swr on non-Windows platforms is always
done for 64 bit archs. For instance in an Gentoo multiarch installation if
swr is enabled, it will be build for all archs.

Fixes: fa4ab7910e3492b09b40e00c0b82a7bb1bae03d0
   swr/rast: Add some SIMD_T utility functors

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=106401
Signed-off-by: Gert Wollny <gw.fossdev at gmail.com>
 src/gallium/drivers/swr/rasterizer/common/simdlib.hpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/gallium/drivers/swr/rasterizer/common/simdlib.hpp b/src/gallium/drivers/swr/rasterizer/common/simdlib.hpp
index 24cf27d4db..a418c2cd6a 100644
--- a/src/gallium/drivers/swr/rasterizer/common/simdlib.hpp
+++ b/src/gallium/drivers/swr/rasterizer/common/simdlib.hpp
@@ -606,7 +606,7 @@ struct SIMDVecHash
     INLINE uint32_t operator ()(Integer<SIMD_T> val) const
-#if defined(_WIN64) || !defined(_WIN32) // assume non-Windows is always 64-bit
+#if defined(_WIN64) || defined(__x86_64__)
         static_assert(sizeof(void*) == 8, "This path only meant for 64-bit code");
         uint64_t crc32 = 0;

PS: I have no commit rights.

More information about the mesa-dev mailing list