[FFmpeg-cvslog] r14981 - trunk/libavcodec/i386/mathops.h

voroshil subversion
Tue Aug 26 21:38:17 CEST 2008


Author: voroshil
Date: Tue Aug 26 21:38:17 2008
New Revision: 14981

Log:
Add explicit (int) cast to i386 optimized MUL* macros.
Wrong result is returned when 16-bit value is passed as value.
Also fixes "Warning: using `%edx' instead of `%dx' due to `l' suffix".


Modified:
   trunk/libavcodec/i386/mathops.h

Modified: trunk/libavcodec/i386/mathops.h
==============================================================================
--- trunk/libavcodec/i386/mathops.h	(original)
+++ trunk/libavcodec/i386/mathops.h	Tue Aug 26 21:38:17 2008
@@ -28,18 +28,18 @@
             "imull %3               \n\t"\
             "shrdl %4, %%edx, %%eax \n\t"\
             : "=a"(rt), "=d"(dummy)\
-            : "a" (ra), "rm" (rb), "i"(FRAC_BITS));\
+            : "a" ((int)ra), "rm" ((int)rb), "i"(FRAC_BITS));\
          rt; })
 #endif
 
 #define MULH(ra, rb) \
     ({ int rt, dummy;\
-     asm ("imull %3\n\t" : "=d"(rt), "=a"(dummy): "a" (ra), "rm" (rb));\
+     asm ("imull %3\n\t" : "=d"(rt), "=a"(dummy): "a" ((int)ra), "rm" ((int)rb));\
      rt; })
 
 #define MUL64(ra, rb) \
     ({ int64_t rt;\
-     asm ("imull %2\n\t" : "=A"(rt) : "a" (ra), "g" (rb));\
+     asm ("imull %2\n\t" : "=A"(rt) : "a" ((int)ra), "g" ((int)rb));\
      rt; })
 
 #endif /* FFMPEG_I386_MATHOPS_H */




More information about the ffmpeg-cvslog mailing list