[Pixman] [PATCH] mmx: Use MMX2 intrinsics from xmmintrin.h directly.

[Pekka Paalanen]

On Sun, 11 Oct 2015 20:59:41 -0700
Matt Turner <mattst88 at gmail.com> wrote:

> We had lots of hacks to handle the inability to include xmmintrin.h
> without compiling with -msse (lest SSE instructions be used in
> pixman-mmx.c). Some recent version of gcc relaxed this restriction.
> 
> Change configure.ac to test that xmmintrin.h can be included and that we
> can use some intrinsics from it, and remove the work-around code from
> pixman-mmx.c.
> 
> Evidently allows gcc 4.9.3 to optimize better as well:
> 
>    text	   data	    bss	    dec	    hex	filename
>  657078	  30848	    680	 688606	  a81de	libpixman-1.so.0.33.3 before
>  656710	  30848	    680	 688238	  a806e	libpixman-1.so.0.33.3 after
> 
> Signed-off-by: Matt Turner <mattst88 at gmail.com>
> ---
> Looks like _MM_SHUFFLE isn't defined by ARM's mmintrin.h.
> 
>  configure.ac        | 15 ++++---------
>  pixman/pixman-mmx.c | 64 ++++-------------------------------------------------
>  2 files changed, 8 insertions(+), 71 deletions(-)

Hi,

tested to build without failures on:
- rpi1 (armv6l), gcc 4.6.3
- x86_64, gcc 4.7.4

Tested-by: Pekka Paalanen <pekka.paalanen at collabora.co.uk>


Thanks,
pq
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 811 bytes
Desc: OpenPGP digital signature
URL: <http://lists.freedesktop.org/archives/pixman/attachments/20151013/2aa3f63e/attachment.sig>