[Pixman] [PATCH 2/3] Moving helper functions in pixman-sse2.c to pixman-sse2.h.

Matt Turner mattst88 at gmail.com
Tue Jan 22 00:49:39 UTC 2019

On Thu, Jan 17, 2019 at 12:27 AM Chris Wilson <chris at chris-wilson.co.uk> wrote:
> Quoting Raghuveer Devulapalli (2019-01-17 00:59:59)
> > From: raghuveer devulapalli <raghuveer.devulapalli at intel.com>
> >
> > These helper function will be reused in pixman-avx2.c implementations in
> > the future.
> Are we ever going to run into a naming conflict in the future? Is it
> worth prefixing all the inlines with sse2_? Probably makes sense so that
> we can see the instruction set used when mixing later.
> -Chris

The SSE2 intrinsics will actually be compiled into VEX-prefixed (AVX)
instructions operating on xmm registers when -mavx2 is used.

I can't think of a reason the lack of a prefix would cause any
confusion for the ones that already have "128" in the name. For
unpack_565_to_8888, etc, maybe it would be best to add a _128 suffix.
We have functions (e.g., pack_565_2x128_128) that look like that

More information about the Pixman mailing list