[vlc-devel] [PATCH] yadif : Add SSSE3 and SSE2 support. porting from FFmpeg.

[Rémi Denis-Courmont]

Le dimanche 27 novembre 2011 02:19:17 Naohiro KORIYAMA, vous avez écrit :
> 2011/11/27 Naohiro KORIYAMA <nkoriyama at gmail.com>:
> > 2011/11/26 Jean-Baptiste Kempf <jb at videolan.org>:
> >> However, we have a yadif filter that is largely tested, and seeing how
> >> costly it is in terms of CPU, any speed improvement, notably with newer
> >> CPUs is very very welcome. Especially if it is usable at 1080i.
> >> 
> >> So, please fix the clobber list and apply.
> > 
> > I changed the clobber list.
> > But I'm not sure how to treat with pb_1, pw_1 at yadif_complete.h.
> > I added these variables to the clobber list and can compile it. Also
> > Filtering using yadif deinterlacer is no problem.
> > But I'm not so familiar with the gcc assemblar, so I want to hear my
> > modification is OK or NG.
> 
> I completely misunderstood about the clobber list. I don't have to
> care about pb_1 and pw_1.
> So, this patch is maybe OK for the clobber list issue.

The clobber list seems fine.

Yadif still has this oddity I pointed out yesterday, but it was there already 
without your patch.

-- 
Rémi Denis-Courmont
http://www.remlab.net/
http://fi.linkedin.com/in/remidenis