[FFmpeg-devel] avfilter/vf_interlace : add checkasm for lowpass_line and AVX2 version

Thomas Mundt tmundt75 at gmail.com
Mon Dec 18 14:59:05 EET 2017


2017-12-18 11:34 GMT+01:00 Martin Vignali <martin.vignali at gmail.com>:

> > >
> >
> > Please also add the changes you made in patch 1 and avx2 to
> vf_tinterlace.
> >
> >
> >
> > For patch 1, IMHO, it's not necessary (the modification is mainly to make
> checkasm test easier to write, and like vf_interlace and vf_tinterlace use
> the same asm
> only one is useful for checkasm)
>

I just thought it might be easier to maintain when code in both filters is
more similar.
But it´s not a big thing, so OK.

In attach new patchs, adding AVX2 lowpass_line 8 and 16 for vf_tinterlace
>

LGTM, thanks.
Do you think, that the complex low pass filter could also be improved that
way?

Regards,
Thomas


More information about the ffmpeg-devel mailing list