[FFmpeg-devel] [PATCH] x86/vc1dsp: Port vc1_*_hor_16b_shift2 to NASM format
Christophe Gisquet
christophe.gisquet at gmail.com
Sun Feb 14 13:21:57 CET 2016
Hi,
2016-02-14 6:49 GMT+01:00 Timothy Gu <timothygu99 at gmail.com>:
> %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
> +; when more functions are ported.
I would still recommend only porting the ones needed in this commit,
but I'm not going to bother you further with this.
> ; Compute the rounder 32-r or 8-r and unpacks it to m7
> %macro LOAD_ROUNDER_MMX 1 ; round
> - movd m7, %1
> + movh m7, %1
Same here: until there's a SSE2 version for some of those functions,
which will require specific code (like a shufps), that's unrelated.
> +cglobal vc1_%2_hor_16b_shift2, 4, 5, 0, dst, stride, src, rnd, h
Do you intend to port more? Because in that case, having a SSE2
version of this one would be possible.
And the wider version would be straightforward in SSE2.
And that would allow you to put at last a (c) in this code you've
spent so much time with.
Otherwise looks OK, but I haven't tested.
--
Christophe
More information about the ffmpeg-devel
mailing list