[FFmpeg-devel] MMX version for put_no_rnd_h264_chroma_mc8_c
Christophe GISQUET
christophe.gisquet
Sun Nov 18 19:02:39 CET 2007
Michael Niedermayer a ?crit :
> less code duplication is preferred unless that would cause a speed loss ...
> if it does, i dont know, but i wont accept duplicating code without
> someone providing benchmark scores showing its faster
The function with rounding is located in dsputil_h264_template_mmx.c
As a consequence, it's templated, and only one version is of interest.
The attached patch is a demonstration of what I'm doing to allow not to
round. I haven't thought yet on how to do it more cleanly...
Benchmarking the old and new codes on an h264 sequence yield no difference.
Best regards,
--
Christophe GISQUET
-------------- next part --------------
A non-text attachment was scrubbed...
Name: h264.diff
Type: text/x-patch
Size: 6507 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20071118/784b0575/attachment.bin>
More information about the ffmpeg-devel
mailing list