[MPlayer-dev-eng] [PATCH] SSE version of DCT64 for mp3lib

[Zuxy Meng]

Hi,

2006/6/23, Guillaume POIRIER <poirierg at gmail.com>:
> Hi,
>
> This was obviously run on a unbuilt tree.
>
> On IA32:
>
> make clean
> rm -f *~ *.o *.a test1 test2
> guillaume at moonlight:~/mplayer/mp3lib$ make
> /usr/lib/gcc-snapshot/bin/gcc -c -I.. -I../libvo -I../../libvo
> -fno-PIC -O4 -march=pentium-m -mtune=pentium-m -pipe -ffast-math
> -fomit-frame-pointer -D_REENTRANT -D_LARGEFILE_SOURCE
> -D_FILE_OFFSET_BITS=64   -o sr1.o sr1.c
> /usr/lib/gcc-snapshot/bin/gcc -c -I.. -I../libvo -I../../libvo
> -fno-PIC -O4 -march=pentium-m -mtune=pentium-m -pipe -ffast-math
> -fomit-frame-pointer -D_REENTRANT -D_LARGEFILE_SOURCE
> -D_FILE_OFFSET_BITS=64   -fomit-frame-pointer -o decode_i586.o
> decode_i586.c
> /usr/lib/gcc-snapshot/bin/gcc -c -I.. -I../libvo -I../../libvo
> -fno-PIC -O4 -march=pentium-m -mtune=pentium-m -pipe -ffast-math
> -fomit-frame-pointer -D_REENTRANT -D_LARGEFILE_SOURCE
> -D_FILE_OFFSET_BITS=64   -o decode_MMX.o decode_MMX.c
> /usr/lib/gcc-snapshot/bin/gcc -c -I.. -I../libvo -I../../libvo
> -fno-PIC -O4 -march=pentium-m -mtune=pentium-m -pipe -ffast-math
> -fomit-frame-pointer -D_REENTRANT -D_LARGEFILE_SOURCE
> -D_FILE_OFFSET_BITS=64   -o dct64_MMX.o dct64_MMX.c
> {standard input}: Assembler messages:
> {standard input}:14: Warning: missing operand; zero assumed
> /usr/lib/gcc-snapshot/bin/gcc -c -I.. -I../libvo -I../../libvo
> -fno-PIC -O4 -march=pentium-m -mtune=pentium-m -pipe -ffast-math
> -fomit-frame-pointer -D_REENTRANT -D_LARGEFILE_SOURCE
> -D_FILE_OFFSET_BITS=64   -o tabinit_MMX.o tabinit_MMX.c
> /usr/lib/gcc-snapshot/bin/gcc -c -I.. -I../libvo -I../../libvo
> -fno-PIC -O4 -march=pentium-m -mtune=pentium-m -pipe -ffast-math
> -fomit-frame-pointer -D_REENTRANT -D_LARGEFILE_SOURCE
> -D_FILE_OFFSET_BITS=64   -o dct36_3dnow.o dct36_3dnow.c
> /usr/lib/gcc-snapshot/bin/gcc -c -I.. -I../libvo -I../../libvo
> -fno-PIC -O4 -march=pentium-m -mtune=pentium-m -pipe -ffast-math
> -fomit-frame-pointer -D_REENTRANT -D_LARGEFILE_SOURCE
> -D_FILE_OFFSET_BITS=64   -o dct64_3dnow.o dct64_3dnow.c
> {standard input}: Assembler messages:
> {standard input}:13: Warning: missing operand; zero assumed
> /usr/lib/gcc-snapshot/bin/gcc -c -I.. -I../libvo -I../../libvo
> -fno-PIC -O4 -march=pentium-m -mtune=pentium-m -pipe -ffast-math
> -fomit-frame-pointer -D_REENTRANT -D_LARGEFILE_SOURCE
> -D_FILE_OFFSET_BITS=64   -o dct36_k7.o dct36_k7.c
> /usr/lib/gcc-snapshot/bin/gcc -c -I.. -I../libvo -I../../libvo
> -fno-PIC -O4 -march=pentium-m -mtune=pentium-m -pipe -ffast-math
> -fomit-frame-pointer -D_REENTRANT -D_LARGEFILE_SOURCE
> -D_FILE_OFFSET_BITS=64   -o dct64_k7.o dct64_k7.c
> {standard input}: Assembler messages:
> {standard input}:13: Warning: missing operand; zero assumed
> ar r libMP3.a sr1.o decode_i586.o decode_MMX.o dct64_MMX.o
> tabinit_MMX.o dct36_3dnow.o dct64_3dnow.o dct36_k7.o dct64_k7.oar:
> creating libMP3.a
> true libMP3.a
> guillaume at moonlight:~/mplayer/mp3lib$
>
> So it looks like things could greatly improved on AMD64.
>

In sr1.c:

#ifdef ARCH_X86
#define CAN_COMPILE_X86_ASM
#endif

may be changed to

#if defined(ARCH_X86) || defined(ARCH_X86_64)
#define CAN_COMPILE_X86_ASM
#endif

-- 
Zuxy
Beauty is truth,
While truth is beauty.
PGP KeyID: E8555ED6