[FFmpeg-devel] [PATCH] split-radix FFT

[Michael Niedermayer]

On Tue, Jul 29, 2008 at 04:46:22PM +0200, Michael Niedermayer wrote:
> On Tue, Jul 29, 2008 at 05:32:44PM +0300, Uoti Urpala wrote:
> > On Tue, 2008-07-29 at 16:18 +0200, Michael Niedermayer wrote:
> > > On Tue, Jul 29, 2008 at 07:36:06AM -0600, Loren Merritt wrote:
> > > > On Tue, 29 Jul 2008, Michael Niedermayer wrote:
> > > > >
> > > > > Also if i use MANGLE to access the 2 root2 globals things clear up a bit in
> > > > > the code generated, so i would suggest that to be done at least, even if
> > > > > the asm stays split up.
> > > > 
> > > > If we don't actually require the output to be position independent, isn't 
> > > > the simpler solution to not compile with -fPIC? Or are we doing some sort 
> > > > of informed trade-off where only the most cpu-intensive functions get 
> > > > textrels?
> > > 
> > > Thats a good argument as well.
> > 
> > Funny how you didn't find the argument good at all when I mentioned it
> > while talking about MANGLE earlier... :)
> 
> You argued in favor of using MANGLE ?
> I distinctly remember you arguing in favor of replacing all MANGLE by
> "m". Which really is the opposite ...

And just to clarify, yes what i considered a good argument was the sentance
above where my reply is. That is to use MANGLE in speed critical code.
That way most textrels are avoided while minimizing the speed impact.

I do not think you ever argued for that. I remember you strongly arguing
toward replacing all MANGLE by "m" knowing that it would break gcc 2.95
and not really caring that it would slow down code compiled with -fPIC.

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Those who are too smart to engage in politics are punished by being
governed by those who are dumber. -- Plato 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080729/f3202633/attachment.pgp>