Re: [Pixman] [PATCH 02/12] vmx: add helper functions

2015-07-15 Thread Oded Gabbay
On Tue, Jul 14, 2015 at 11:40 AM, Siarhei Siamashka siarhei.siamas...@gmail.com wrote: Acked-by: Siarhei Siamashka siarhei.siamas...@gmail.com Just one question about the code. There are some places where both big and little endian variants are identical but we still have an ifdef: +static

Re: [Pixman] [PATCH 02/12] vmx: add helper functions

2015-07-14 Thread Siarhei Siamashka
On Thu, 2 Jul 2015 13:04:07 +0300 Oded Gabbay oded.gab...@gmail.com wrote: This patch adds the following helper functions for reuse of code, hiding BE/LE differences and maintainability. All of the functions were defined as static force_inline. Names were copied from pixman-sse2.c so

[Pixman] [PATCH 02/12] vmx: add helper functions

2015-07-02 Thread Oded Gabbay
This patch adds the following helper functions for reuse of code, hiding BE/LE differences and maintainability. All of the functions were defined as static force_inline. Names were copied from pixman-sse2.c so conversion of fast-paths between sse2 and vmx would be easier from now on. Therefore,