Re: [FFmpeg-devel] [PATCH] x86/vc1dsp: Port vc1_*_hor_16b_shift2 to NASM format

2016-02-14 Thread Timothy Gu
On Sun, Feb 14, 2016 at 01:21:57PM +0100, Christophe Gisquet wrote: > Hi, > > 2016-02-14 6:49 GMT+01:00 Timothy Gu : > > %if HAVE_MMX_INLINE > > Isn't that macro meant for C code (and in config.asm without much of a > purpose)? Yes, but this code isn't used unless inline asm is enabled so I don

Re: [FFmpeg-devel] [PATCH] x86/vc1dsp: Port vc1_*_hor_16b_shift2 to NASM format

2016-02-14 Thread Christophe Gisquet
Hi, 2016-02-14 6:49 GMT+01:00 Timothy Gu : > %if HAVE_MMX_INLINE Isn't that macro meant for C code (and in config.asm without much of a purpose)? I suspect it is not useful, but I haven't dug into that. > +; XXX some of these macros are not used right now, but they will in the > future > +;

[FFmpeg-devel] [PATCH] x86/vc1dsp: Port vc1_*_hor_16b_shift2 to NASM format

2016-02-13 Thread Timothy Gu
--- libavcodec/x86/vc1dsp.asm | 92 - libavcodec/x86/vc1dsp_mmx.c | 61 -- 2 files changed, 99 insertions(+), 54 deletions(-) diff --git a/libavcodec/x86/vc1dsp.asm b/libavcodec/x86/vc1dsp.asm index 91a1991..2e9f067 100644