[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