[FFmpeg-devel] [PATCH] Roundup issue #301

Christophe GISQUET christophe.gisquet
Fri Dec 28 19:22:18 CET 2007


Hi,

This addresses roundup issue #301 :
https://roundup.mplayerhq.hu/roundup/ffmpeg/issue301

Summary of changes:
- Use of MANGLE when loading into mmx registers some constants
- Conversion to non-static for those constants and thus renaming with
  ff_prefix
- Removal of last parameter of MSPEL_FILTER13_CORE (was constant)
- Use of "+r" instead of stricter but unnecessary "+g"
- Use of REG_c and direct loading of some of the above (not tested

No speed regression is noticeable with the new code, but the performance
vastly varies depending on the compiler. For a 720p test sequence:
2.95   ~ 4.1s
3.3.6  ~ 3.8s  (3.2 in 64b)
4.2.1  ~ 3.4s  (3.1 in 64b)

Considering those numbers, I think I'll pass next time.

(And yeah! for hard-coded limitations)

Best regards,
-- 
Christophe GISQUET

-------------- next part --------------
A non-text attachment was scrubbed...
Name: vc1_gcc295.diff
Type: text/x-patch
Size: 14407 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20071228/8b9afa86/attachment.bin>



More information about the ffmpeg-devel mailing list