[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