[MPlayer-dev-eng] vf_yadif.c: fix for (probably) older binutils

Reimar Döffinger Reimar.Doeffinger at stud.uni-karlsruhe.de
Sun Oct 1 16:42:12 CEST 2006


Hello,
On Thu, Sep 21, 2006 at 11:17:55PM +0200, Tomas Janousek wrote:
> my as does not understand "-1-1(something)", so I changed it to "0-1-1" 
> and so on.
> 
> I have binutils-2.15.90.0.3-5 on Fedora Core 2.

So is this patch or the attached one or something completely different preferable?

Greetings,
Reimar Döffinger
-------------- next part --------------
Index: libmpcodecs/vf_yadif.c
===================================================================
--- libmpcodecs/vf_yadif.c	(revision 20011)
+++ libmpcodecs/vf_yadif.c	(working copy)
@@ -83,8 +83,8 @@
             "pmaxsw   "#tmp", "#dst" \n\t"
 
 #define CHECK(pj,mj) \
-            "movq "#pj"-1(%[cur],%[mrefs]), %%mm2 \n\t" /* cur[x-refs-1+j] */\
-            "movq "#mj"-1(%[cur],%[prefs]), %%mm3 \n\t" /* cur[x+refs-1-j] */\
+            "movq "#pj"(%[cur],%[mrefs]), %%mm2 \n\t" /* cur[x-refs-1+j] */\
+            "movq "#mj"(%[cur],%[prefs]), %%mm3 \n\t" /* cur[x+refs-1-j] */\
             "movq      %%mm2, %%mm4 \n\t"\
             "movq      %%mm2, %%mm5 \n\t"\
             "pxor      %%mm3, %%mm4 \n\t"\
@@ -193,13 +193,13 @@
             "paddw     %%mm3, %%mm0 \n\t"\
             "psubw    %[pw1], %%mm0 \n\t" /* spatial_score */\
 \
-            CHECK(-1,1)\
+            CHECK(-2,0)\
             CHECK1\
-            CHECK(-2,2)\
+            CHECK(-3,1)\
             CHECK2\
-            CHECK(1,-1)\
+            CHECK(0,-2)\
             CHECK1\
-            CHECK(2,-2)\
+            CHECK(1,-3)\
             CHECK2\
 \
             /* if(p->mode<2) ... */\


More information about the MPlayer-dev-eng mailing list