Re: PATCH: Add SSE4.1 support

[Uros Bizjak <ubizjak at gmail dot com>]

H. J. Lu wrote:

> I am testing the following patch. I will check it on after bootstrap
> and test are finished without regressions on Linux/ia32 and
> Linux/Intel64.
>
>
>  /* Test that {,x,e,p,t,a}mmintrin.h, mm3dnow.h and mm_malloc.h are
>     usable with -O -pedantic-errors.  */
>  /* { dg-do compile { target i?86-*-* x86_64-*-* } } */
> -/* { dg-options "-O -pedantic-errors -march=k8 -m3dnow -mssse3 -msse4a" } */
> +/* { dg-options "-O -pedantic-errors -march=k8 -m3dnow -mssse3 -msse4.1 -msse4a" } */
>  
>  #include <ammintrin.h>
> +#include <smmintrin.h>
>  #include <tmmintrin.h>
>  #include <mm3dnow.h>
>   
smmintrin.h already includes tmmintrin.h, so it is enough to include 
smmintrin.h _instead_ of tmmintrin.h to include full chain of SSE 
intrinsic headers. Also, IIRC -msse4.1 implies -mssse3, so it is enough 
to add -msse4.1 _instead_ of -mssse3. Oh, and a comment should be 
updated, as new header is included.

Patch is OK with this change to all tests. But be aware that these four 
tests are the ones that test your headers ;)

Uros.