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

[Michael Niedermayer]

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 also think this might fail on some platforms with some gcc versions

Otherwise it should be ok if its tested and works

[...]

--
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Those who are best at talking, realize last or never when they are wrong.
-------------- 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/20101130/0405c1b5/attachment.pgp>