[Mesa-dev] [PATCH] i965: Compile with -msse3

[Kenneth Graunke]

On Friday, July 14, 2017 12:19:39 PM PDT Matt Turner wrote:
> All CPUs that can be paired with a GPU supported by i965_dri.so supports
> SSE3. This allows us to ensure that some vectorized version of the tiled
> memcpy path is enabled on 32-bit systems.
> 
> This also ensures that __builtin_ia32_clflush is always usable.
> 
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=101774
> Tested-by: Tobias Klausmann <tobias.johannes.klausmann at mni.thm.de>
> ---
>  src/mesa/drivers/dri/i965/Makefile.am | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/src/mesa/drivers/dri/i965/Makefile.am b/src/mesa/drivers/dri/i965/Makefile.am
> index ecc1e766f1..9293820c21 100644
> --- a/src/mesa/drivers/dri/i965/Makefile.am
> +++ b/src/mesa/drivers/dri/i965/Makefile.am
> @@ -42,7 +42,8 @@ AM_CFLAGS = \
>  	$(DEFINES) \
>  	$(VISIBILITY_CFLAGS) \
>  	$(LIBDRM_CFLAGS) \
> -	$(VALGRIND_CFLAGS)
> +	$(VALGRIND_CFLAGS) \
> +	-msse3
>  
>  AM_CXXFLAGS = $(AM_CFLAGS)
>  
> 

Looks like the Vulkan driver already sets -msse2, so we at least need to
do that in i965...and your rationale for "everything also does SSE3, so
why not" makes sense to me.

Acked-by: Kenneth Graunke <kenneth at whitecape.org>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: This is a digitally signed message part.
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170714/843bd6e9/attachment.sig>