[Ffmpeg-cvslog] r8012 - trunk/libavcodec/i386/dsputil_h264_template_mmx.c

[Michael Niedermayer]

Hi

On Sun, Feb 18, 2007 at 03:11:27PM +0000, M?ns Rullg?rd wrote:
> michael <subversion at mplayerhq.hu> writes:
> 
> > Author: michael
> > Date: Sun Feb 18 00:43:02 2007
> > New Revision: 8012
> >
> > Modified:
> >    trunk/libavcodec/i386/dsputil_h264_template_mmx.c
> >
> > Log:
> > rewrite H264_CHROMA_MC4_TMPL (20% faster)
> 
> This broke x86-64.
> 
> > +        "add %3, %1                 \n\t"
> [...]
> > +        : "+r"(dst), "+r"(src), "+r"(h)
> > +        : "r"(stride), "m"(ff_pw_32), "m"(x), "m"(y), "m"(ff_pw_8)
> 
> The above add, and a few others, compile to "add %edx, %rsi", which
> the assembler doesn't like at all, the operands having different
> sizes.  This patch makes the code compile, but I'll leave it to the
> gurus to decide on the correct fix:

patch looks ok

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

It is dangerous to be right in matters on which the established authorities
are wrong. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20070218/629f9546/attachment.pgp>