[FFmpeg-devel] [PATCH] use 'l' asm suffix to make icc happy

Reimar Döffinger Reimar.Doeffinger
Sat Dec 15 22:25:37 CET 2007


Hello,
the operands to dec in these two cases can be a memory address, so the
assembler can not really infer the size, and icc fails to compile.
Attached patch fixes it.

Greetings,
Reimar D?ffinger
-------------- next part --------------
Index: libavcodec/i386/vc1dsp_mmx.c
===================================================================
--- libavcodec/i386/vc1dsp_mmx.c	(revision 11225)
+++ libavcodec/i386/vc1dsp_mmx.c	(working copy)
@@ -95,7 +95,7 @@
         SHIFT2_LINE(168, 4, 1, 2, 3)
         "sub       %8, %1                  \n\t"
         "add       $8, %2                  \n\t"
-        "dec       %0                      \n\t"
+        "decl      %0                      \n\t"
         "jnz 1b                            \n\t"
         : "+g"(w), "+r"(src), "+r"(dst)
         : "r"(stride), "r"(-2*stride), "m"(shift),
@@ -191,7 +191,7 @@
         TRANSFER_DO_PACK
         "add       %7, %1                  \n\t"
         "add       %5, %2                  \n\t"
-        "dec       %0                      \n\t"
+        "decl      %0                      \n\t"
         "jnz 1b                            \n\t"
         : "+g"(h), "+r"(src),  "+r"(dst)
         : "r"(offset), "r"(-2*offset), "g"(stride), "m"(rnd),



More information about the ffmpeg-devel mailing list