[FFmpeg-devel] [PATCH] stricter constraints for libavcodec/i386/mpegvideo_mmx.c

Michael Niedermayer michaelni
Sat Aug 23 03:46:09 CEST 2008


On Fri, Aug 22, 2008 at 01:15:36PM +0200, Alexis Ballier wrote:
> Hi,
> 
> All these variables are used as left operands of a movd instruction,
> which does accept only memory or register operands while the "g"
> constraint also allows immediates. This patch shall reflect this by
> using "mr" instead.

I would prefer "rm" over "mr" besides
please provide START/STOP_TIMER benchmarks of each changed function
the gcc output from the errors
and if possible some information of what, if anything changes in the
asm output

yes, the functions are rather important to performance and i dont want
changes to, not even bugfixes withot understandig what effect these
changes have to performance.

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Dictatorship naturally arises out of democracy, and the most aggravated
form of tyranny and slavery out of the most extreme liberty. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080823/a760ac21/attachment.pgp>



More information about the ffmpeg-devel mailing list