[MPlayer-dev-eng] [PATCH] x86_64 mmx/sse/3dnow optimisation support

[Jan Killius]

On Friday 22 October 2004 01:38, Aurelien Jacobs wrote:
> On Fri, 22 Oct 2004 00:21:55 +0300
>
> Jan Knutar <jknutar at nic.fi> wrote:
> > On Thursday 21 October 2004 14:58, Aurelien Jacobs wrote:
> > > On Sun, 17 Oct 2004 22:40:51 -0400
> > >
> > > D Richard Felker III <dalias at aerifal.cx> wrote:
> > > > ok, this all sounds much better. i haven't read the patch yet but
> > > > from what you've said i think it's good so far.
> > >
> > > Commited.
> > > Hope it won't break anything !
> >
> > Perhaps related: 
> > http://www.mplayerhq.hu/pipermail/mplayer-advusers/2004-October/001043.ht
> >ml
>
> Unfortunatly it is related !
> Here is a patch that fix this crash. Unfortunatly, I don't have
> my amd64 system handy, so I won't be able to test it on amd64
> during the next 18 hours :-(
> It would be nice if someone on amd64 could test it (the
> important point is to test -vf scale).
Can't build mplayer on my machine:
cc -c -I../libvo -I../../libvo -I/usr/X11R6/include -O4   -pipe -ffast-math 
-fomit-frame-pointer -D_REENTRANT -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64   
-I. -I..  -I.. -o swscale.o swscale.c
In file included from swscale.c:748:
swscale_template.c: In function `swScale_MMX2':
swscale_template.c:2754: warning: cast from pointer to integer of different 
size
swscale_template.c:2761: warning: cast from pointer to integer of different 
size
swscale_template.c:2133: warning: use of memory input without lvalue in asm 
operand 4 is deprecated
swscale_template.c:2300: warning: use of memory input without lvalue in asm 
operand 3 is deprecated
swscale_template.c:2300: warning: use of memory input without lvalue in asm 
operand 4 is deprecated
swscale_template.c:2133: warning: use of memory input without lvalue in asm 
operand 4 is deprecated
swscale_template.c:2133: warning: use of memory input without lvalue in asm 
operand 4 is deprecated
swscale_template.c:2481: warning: use of memory input without lvalue in asm 
operand 2 is deprecated
swscale_template.c:2481: warning: use of memory input without lvalue in asm 
operand 3 is deprecated
swscale_template.c:2481: warning: use of memory input without lvalue in asm 
operand 4 is deprecated
swscale_template.c:2133: warning: use of memory input without lvalue in asm 
operand 4 is deprecated
swscale_template.c:2300: warning: use of memory input without lvalue in asm 
operand 3 is deprecated
swscale_template.c:2300: warning: use of memory input without lvalue in asm 
operand 4 is deprecated
swscale_template.c:2133: warning: use of memory input without lvalue in asm 
operand 4 is deprecated
swscale_template.c:2133: warning: use of memory input without lvalue in asm 
operand 4 is deprecated
swscale_template.c:2481: warning: use of memory input without lvalue in asm 
operand 2 is deprecated
swscale_template.c:2481: warning: use of memory input without lvalue in asm 
operand 3 is deprecated
swscale_template.c:2481: warning: use of memory input without lvalue in asm 
operand 4 is deprecated
swscale_template.c:765: warning: use of memory input without lvalue in asm 
operand 2 is deprecated
swscale_template.c:772: warning: use of memory input without lvalue in asm 
operand 2 is deprecated
swscale_template.c:780: warning: use of memory input without lvalue in asm 
operand 2 is deprecated
{standard input}: Assembler messages:
{standard input}:5722: Error: suffix or operands invalid for `movzwl'
{standard input}:5723: Error: suffix or operands invalid for `movzwl'
{standard input}:5955: Error: suffix or operands invalid for `movzwl'
{standard input}:5956: Error: suffix or operands invalid for `movzwl'
{standard input}:6027: Error: suffix or operands invalid for `movzwl'
{standard input}:6028: Error: suffix or operands invalid for `movzwl'
{standard input}:6607: Error: suffix or operands invalid for `movzwl'
{standard input}:6608: Error: suffix or operands invalid for `movzwl'
{standard input}:6792: Error: suffix or operands invalid for `movzwl'
{standard input}:6793: Error: suffix or operands invalid for `movzwl'
{standard input}:7129: Error: suffix or operands invalid for `movzwl'
{standard input}:7130: Error: suffix or operands invalid for `movzwl'
{standard input}:7175: Error: suffix or operands invalid for `movzwl'
{standard input}:7176: Error: suffix or operands invalid for `movzwl'
{standard input}:7337: Error: suffix or operands invalid for `movzwl'
{standard input}:7338: Error: suffix or operands invalid for `movzwl'
{standard input}:7393: Error: suffix or operands invalid for `movzwl'
{standard input}:7394: Error: suffix or operands invalid for `movzwl'
{standard input}:8135: Error: suffix or operands invalid for `movzwl'
{standard input}:8136: Error: suffix or operands invalid for `movzwl'
{standard input}:8462: Error: suffix or operands invalid for `movzwl'
{standard input}:8463: Error: suffix or operands invalid for `movzwl'
{standard input}:8564: Error: suffix or operands invalid for `movzwl'
{standard input}:8565: Error: suffix or operands invalid for `movzwl'
{standard input}:8636: Error: suffix or operands invalid for `movzwl'
{standard input}:8637: Error: suffix or operands invalid for `movzwl'
{standard input}:8830: Error: suffix or operands invalid for `movzwl'
{standard input}:8831: Error: suffix or operands invalid for `movzwl'
{standard input}:9107: Error: suffix or operands invalid for `movzwl'
{standard input}:9108: Error: suffix or operands invalid for `movzwl'
{standard input}:9154: Error: suffix or operands invalid for `movzwl'
{standard input}:9155: Error: suffix or operands invalid for `movzwl'
{standard input}:9461: Error: suffix or operands invalid for `movzwl'
{standard input}:9462: Error: suffix or operands invalid for `movzwl'
{standard input}:9518: Error: suffix or operands invalid for `movzwl'
{standard input}:9519: Error: suffix or operands invalid for `movzwl'
make[1]: *** [swscale.o] Error 1
make[1]: Leaving directory `/usr/local/src/main/postproc'
make: *** [postproc/libswscale.a] Error 2

>
> I don't know why I modified this in the first time. This
> probably mean at least 10l for me.
>
> Should I commit this before testing on amd64 or should I wait ?
>
> Aurel

-- 
        Jan