Re: [Pixman] [Patch] Fix pixman build with clang

2017-12-12 Thread Manoj Gupta 
Thanks Emil,

I see that this is indeed fixed in master. There is another problem with
__builtin_shuffle though which clang doesn't support.

On Tue, Dec 12, 2017 at 7:26 AM, Emil Velikov 
wrote:

> On 8 December 2017 at 23:32, Manoj Gupta  wrote:
> > Hi,
> >
> > I am encountering a build issue in pixman when compiling with with clang.
> >
> > pixman-mmx.c:100:20: error: constraint 'K' expects an integer constant
> > expression
> > : "y" (__A), "K" (__N)
> >
> Seems like you guys are building some pretty old stuff ;-)
>
> The issue was resolved upstream back in 2015 [1] and there has been a
> release with said fix [2].
>
> -Emil
>
> [1] https://cgit.freedesktop.org/pixman/commit/?id=
> d24b415f3e2753a588759d028b811e1ce38fea6c
> [2] https://lists.x.org/archives/xorg-announce/2015-December/002666.html
>
___
Pixman mailing list
Pixman@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/pixman

Re: [Pixman] Crash during stress-test

2017-12-12 Thread Emil Velikov 
On 19 November 2017 at 18:26, LE GARREC Vincent
 wrote:
> I made a clone on https://github.com/bansan85/pixman/tree/stress_test_file
> I tried to make lots of small commits to make review easier.

A lot better, thank you. There are some whitespace fixes alongside the
feature ones.
Can you give it another quick look and send the lot to the list for
review - I think the recommended way is via git send-email.

-Emil
___
Pixman mailing list
Pixman@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/pixman

Re: [Pixman] [Patch] Fix pixman build with clang

2017-12-12 Thread Emil Velikov 
On 8 December 2017 at 23:32, Manoj Gupta  wrote:
> Hi,
>
> I am encountering a build issue in pixman when compiling with with clang.
>
> pixman-mmx.c:100:20: error: constraint 'K' expects an integer constant
> expression
> : "y" (__A), "K" (__N)
>
Seems like you guys are building some pretty old stuff ;-)

The issue was resolved upstream back in 2015 [1] and there has been a
release with said fix [2].

-Emil

[1] 
https://cgit.freedesktop.org/pixman/commit/?id=d24b415f3e2753a588759d028b811e1ce38fea6c
[2] https://lists.x.org/archives/xorg-announce/2015-December/002666.html
___
Pixman mailing list
Pixman@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/pixman