[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