[MPlayer-dev-eng] [PATCH] Enable mp3lib's SSE routines on AMD64.

Attila Kinali attila at kinali.ch
Sat May 5 11:28:34 CEST 2007


On Fri, 4 May 2007 17:52:38 +0200
Guillaume Poirier <gpoirier at mplayerhq.hu> wrote:

> Attached patch allows to $SUBJ
> It does so by moving array costab_mmx from decode_MMX.c (which can't be
> compiled on AMD64) to a separate file costab_MMX.c.
> 
> I've played a couple of MP3s on my Hi-Fi stereo and as far as I hear,
> there aren't any artifacts, which isn't surprising.
> 
> I'm sure it can be improved, so I'm interested in hearing from your  
> guys.

Patch works, output is correct (binary compare), but it's slightly
slower on my machine(AMD Athlon(tm) 64 Processor 3700+):

./mplayer -ao pcm:file=/dev/null /tmp/01\ -\ Standing\ in\ the\ Sunset\ Glow.mp3 -benchmark -quiet

w/o patch:
BENCHMARKs: VC:   0.000s VO:   0.000s A:   5.746s Sys:   0.018s =    5.764s

w/ patch:
BENCHMARKs: VC:   0.000s VO:   0.000s A:   5.807s Sys:   0.017s =    5.824s

This is a difference of 1%

			Attila Kinali
-- 
Linux ist... wenn man einfache Dinge auch mit einer kryptischen
post-fix Sprache loesen kann
                        -- Daniel Hottinger



More information about the MPlayer-dev-eng mailing list