[MPlayer-cvslog] CVS: main/liba52 a52_internal.h, 1.4, 1.5 downmix.c, 1.17, 1.18 imdct.c, 1.27, 1.28 resample.c, 1.16, 1.17 resample_mmx.c, 1.17, 1.18

[Guillaume POIRIER]

Hi,

On 8/7/05, Aurelien Jacobs <aurel at gnuage.org> wrote:
> On Sat, 6 Aug 2005 00:43:03 +0200
> Guillaume POIRIER <poirierg at gmail.com> wrote:
> 
> > Hi,
> >
> > On 8/5/05, Aurelien Jacobs CVS <syncmail at mplayerhq.hu> wrote:
> > > CVS change done by Aurelien Jacobs CVS
> > >
> > > Update of /cvsroot/mplayer/main/liba52
> > > In directory mail:/var2/tmp/cvs-serv14971
> > >
> > > Modified Files:
> > >         a52_internal.h downmix.c imdct.c resample.c resample_mmx.c
> > > Log Message:
> > > liba52 asm optimizations ported to amd64
> >
> > This seems to break on IA-32.
> 
> I can't reproduce any problem on ia32...
> Do you have any sample which breaks ? On what CPU ? SSE only
> or 3dnow+SSE ?

My cpu is an AMD64, and the code breaks with any DVD I have. I tried
to narrow down the problem compiling with debugging symbols, but then
mplayer works fine.

I did not try to disable SSE or 3dnow.
uau on IRC says that: "line 816 in imdct.c was the one that caused the
problem (at least i got it working after reverting that one)".

I hope that helps.

Guillaume
-- 
A legend is an old man with a cane known for
what he used to do. I'm still doing it.
  -- Miles Davis