[FFmpeg-devel] [PATCH] yadif sse2/ssse3

Luca Barbato lu_zero
Tue Nov 30 09:33:27 CET 2010


On 11/30/2010 05:52 AM, Baptiste Coudurier wrote:
> Hi Michael,
> 
> On 11/29/10 4:46 PM, Michael Niedermayer wrote:
>> On Sat, Nov 20, 2010 at 06:32:39PM -0800, Baptiste Coudurier wrote:
>>> Hi guys,
>>>
>>> $subject.
>> [...]
>>     > +#if HAVE_SSE
>>> +    DECLARE_ALIGNED(16, uint8_t, tmp0[16]);
>>> +    DECLARE_ALIGNED(16, uint8_t, tmp1[16]);
>>> +    DECLARE_ALIGNED(16, uint8_t, tmp2[16]);
>>> +    DECLARE_ALIGNED(16, uint8_t, tmp3[16]);
>>> +#else
>>>      uint64_t tmp0, tmp1, tmp2, tmp3;
>>> +#endif
>>
>> This seems unneeded (align always should be fine)
> 
> I'm not sure I understand, do you mean I should remove the DECLARE_ALIGNED ?
> 

remove the ifdef and keep the same var declaration (aligned) probably.

lu

-- 

Luca Barbato
Gentoo/linux
http://dev.gentoo.org/~lu_zero




More information about the ffmpeg-devel mailing list