[Ffmpeg-devel] [PATCH] SSE counterpart of ff_imdct_calc_3dn2

[Guillaume Poirier]

Hi,

Diego Biurrun wrote:
> On Thu, Sep 14, 2006 at 11:02:48AM +0200, Guillaume POIRIER wrote:
> 
>>On 9/14/06, Zuxy Meng <zuxy.meng at gmail.com> wrote:
>>
>>>2006/9/13, Benjamin Larsson <banan at student.ltu.se>:
>>>
>>>>Missing?
>>>
>>>I mean the one I sent out last time.
>>
>>Well, it still needs -fomit-frame-pointer to get compiled at all, so I
>>don't think it's good to go, unless we add the requirement that FFmpeg
>>needs to be compiled with -fomit-frame-pointer.
> 
> 
> Wasn't this just on your specific gcc version?

No, I've checked with GCC 3.3, 4.0.3, 4.2.x (from svn) and they all
fail on the same code on IA32.

Note that it works in all cases on AMD64, which is logical as GCC has
2x more registers to play with.

I don't know what's the proper way to fix/work around this problem.
Maybe it would require to disable this code if compiled without
-fomit-frame-pointer, but I have no idea if GCC defines any symbol
that would allow to detect this...

Guillaume