[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