Re: [libav-devel] [PATCH 12/12] x86/yadif-10: remove duplicate ABS macro

2016-10-21 Thread Diego Biurrun
On Sun, Oct 16, 2016 at 08:11:02PM +0200, Janne Grunau wrote:
> From: James Almer 
> 
> And use the x86util ones instead, which are optimized for mmxext/sse2.
> About ~1% increase in performance on pre SSSE3 processors.
> 
> Signed-off-by: James Almer 
> Signed-off-by: Michael Niedermayer 
> Signed-off-by: Janne Grunau 
> ---
>  libavfilter/x86/yadif-10.asm | 21 -
>  1 file changed, 4 insertions(+), 17 deletions(-)

OK

Diego
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel


Re: [libav-devel] [PATCH 12/12] x86/yadif-10: remove duplicate ABS macro

2016-10-17 Thread Vittorio Giovara
On Sun, Oct 16, 2016 at 2:11 PM, Janne Grunau  wrote:
> From: James Almer 
>
> And use the x86util ones instead, which are optimized for mmxext/sse2.
> About ~1% increase in performance on pre SSSE3 processors.

Question unrelated to this patch, is there any other place where it
would make sense to do this in the tree? A quick grep shows there is a
duplicate macro in x86/ac3dsp.asm, butI don't know asm enough to
judge.
Thanks
-- 
Vittorio
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel