[Pixman] [PATCH 02/12] vmx: add helper functions
Oded Gabbay
oded.gabbay at gmail.com
Wed Jul 15 04:52:37 PDT 2015
On Tue, Jul 14, 2015 at 11:40 AM, Siarhei Siamashka
<siarhei.siamashka at gmail.com> wrote:
> Acked-by: Siarhei Siamashka <siarhei.siamashka at 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 force_inline vector unsigned int
>> +expand_pixel_32_1x128 (uint32_t data)
>> +{
>> + vector unsigned int vdata;
>> +
>> + vdata = unpack_32_1x128 (data);
>> +
>> +#ifdef WORDS_BIGENDIAN
>> + return vec_perm (vdata, vdata,
>> + (vector unsigned char)AVV (
>> + 0x08, 0x09, 0x0A, 0x0B, 0x0C, 0x0D, 0x0E, 0x0F,
>> + 0x08, 0x09, 0x0A, 0x0B, 0x0C, 0x0D, 0x0E, 0x0F));
>> +#else
>> + return vec_perm (vdata, vdata,
>> + (vector unsigned char)AVV (
>> + 0x08, 0x09, 0x0A, 0x0B, 0x0C, 0x0D, 0x0E, 0x0F,
>> + 0x08, 0x09, 0x0A, 0x0B, 0x0C, 0x0D, 0x0E, 0x0F));
>> +#endif
>
> Is this intentional?
No, and it is of course erroneous. The reason it slipped under my
radar is because this function is not used at all.
Thanks for catching that :)
I removed it from the original patch.
Oded
More information about the Pixman
mailing list