Re: [Pixman] [BUG pixman] f9c91ee2f27eaea68d8c3a130bf7d4bc0c860834 breaks compilation

2012-07-09 Thread Matt Turner
On Mon, Jul 9, 2012 at 1:55 AM, Knut Petersen wrote: > Søren, the bad commit was supposed to fix a gcc -O0 compile problem, but it > breaks > gcc -O0 compilation here. Reverting f9c91ee2 fixes the problem for me. Is this build automated? If it's an automated build that runs the test suite, you'r

Re: [Pixman] [PATCH] Make pixman-mmx.c compile on x86-32 without optimization

2012-07-09 Thread Matt Turner
On Mon, Jul 9, 2012 at 7:31 AM, Søren Sandmann wrote: > From: Søren Sandmann Pedersen > > When not optimizing, write _mm_shuffle_pi16() as a statement > expression with inline assembly. That way we avoid > __builtin_ia32_pshufw(), which is only available when compiling with > -msse, while still a

Re: [Pixman] [PATCH] Make pixman-mmx.c compile on x86-32 without optimization

2012-07-09 Thread Knut Petersen
That compiles cleanly here. Tested-by: Knut Petersen From: Søren Sandmann Pedersen When not optimizing, write _mm_shuffle_pi16() as a statement expression with inline assembly. That way we avoid __builtin_ia32_pshufw(), which is only available when compiling with -msse, while still allowing

[Pixman] [PATCH] Make pixman-mmx.c compile on x86-32 without optimization

2012-07-09 Thread Søren Sandmann
From: Søren Sandmann Pedersen When not optimizing, write _mm_shuffle_pi16() as a statement expression with inline assembly. That way we avoid __builtin_ia32_pshufw(), which is only available when compiling with -msse, while still allowing the non-optimizing gcc to understand that the second argum

Re: [Pixman] [BUG pixman] f9c91ee2f27eaea68d8c3a130bf7d4bc0c860834 breaks compilation

2012-07-09 Thread Søren Sandmann
Knut Petersen writes: > Søren, the bad commit was supposed to fix a gcc -O0 compile problem, but it > breaks > gcc -O0 compilation here. Reverting f9c91ee2 fixes the problem for me. Thanks for the bug report. Can you please try the following patch and let me know if it fixes the problem? Sore