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

Diego Biurrun diego at biurrun.de
Sat May 5 10:52:34 CEST 2007


On Fri, May 04, 2007 at 05:52:38PM +0200, Guillaume Poirier 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.
> 
> --- mp3lib/costab_MMX.c	(r??vision 0)
> +++ mp3lib/costab_MMX.c	(r??vision 0)
> @@ -0,0 +1,34 @@
> +unsigned long __attribute__((aligned(16))) costab_mmx[] =
> +{
> +	[...]
> +};

You should keep some sort of licensing header, though in this case it's
pretty moot.  Looks good otherwise.

Diego



More information about the MPlayer-dev-eng mailing list