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

Michael Niedermayer michaelni
Wed Dec 1 01:42:18 CET 2010


On Tue, Nov 30, 2010 at 09:33:27AM +0100, Luca Barbato wrote:
> 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.

yes

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have often repented speaking, but never of holding my tongue.
-- Xenocrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20101201/3b1acd9a/attachment.pgp>



More information about the ffmpeg-devel mailing list