[MPlayer-dev-eng] compiling mplayer with gcc 4.6, broken audio decoding

[Jindrich Makovicka]

On Thu, 31 Mar 2011 07:12:34 -0300
Kazuo Teramoto <kaz.rag at gmail.com> wrote:

> On Thu, Mar 31, 2011 at 6:25 AM, Kazuo Teramoto <kaz.rag at gmail.com>
> wrote:
> > But if I change the -O2 to -O0, I don't get the warning AND I don't
> > get the very high pitch sound with mp3lib.
> >
> > So, in summary compiling the mp3lib/dct64_sse.c with gcc 4.6 and
> > -O0, I don't get the high pitch sound with mp3lib.
> >
> 
> I learned about the --save-temps! (sorry about my lack of gcc
> expertise =])
> 
> So I'm attaching the .s of -O1 and -O0 of the dct64_sse.c file compile
> with the gcc 4.6 on my machine.
> 
> PS. Any more flag I'm missing?
> 
> Regards,
> Kazuo.
> 

Can you try this change? It's not a proper fix, but it makes
gcc 4.6 emit complete nnnn table in the output.

Index: dct64_sse.c
===================================================================
--- dct64_sse.c	(revision 33341)
+++ dct64_sse.c	(working copy)
@@ -112,7 +112,6 @@
     }
 
     {
-        real *costab = costab_mmx + 24;
         int i;
 
         __asm__(
@@ -121,7 +120,7 @@
             "movaps    %1, %%xmm5\n\t"
             "movaps    %%xmm5, %%xmm6\n\t"
             :
-            :"m"(*costab), "m"(*nnnn)
+            :"m"(costab_mmx[24]), "m"(*nnnn)
            );
 
         for (i = 0; i < 0x20; i += 8)


Regards,
-- 
Jindrich Makovicka