[FFmpeg-devel] Question about -fPIC usage for some files

[Michael Niedermayer]

On Fri, Feb 08, 2008 at 05:30:50PM +0100, Reimar D?ffinger wrote:
> Hello,
> On Fri, Feb 08, 2008 at 04:26:46PM +0100, Michael Niedermayer wrote:
> > On Fri, Feb 08, 2008 at 03:14:42PM +0100, Thorsten Jordan wrote:
> > > i386/flacdsp_mmx.c: In function ?ff_flac_compute_autocorr_sse2?:
> > > i386/flacdsp_mmx.c:87: error: can't find a register in class
> > > ?GENERAL_REGS? while reloading ?asm?
> > 
> > if any of them need ebx they should be under HAVE_EBX_AVAILABLE
> > if any of them needs 7 registers they should be under HAVE_7REGS
> > the remaining cases are gcc bugs and should be reported to the gcc devels
> > (why do i think they all fall in the same category ...)
> 
> IIRC that last one does fall in that last category and can be fixed with
> a __aatribute__((noinline)), the funniest thing about that is that ICC
> (10.1.008) can not handle that code even without -fPIC, and needs the same hack...

Why dont you change the asm a little bit to be more easy for our old gcc
and co?

"=m"(autoc[j]), "=m"(autoc[j+1]), "=m"(autoc[j+2])
to
"=m"(autoc[j])
and access the other 2 by 8+%X 16+%X

"m"(*ff_pd_1)
to

and MANGLE()

that would be 3 operands less which gcc can stumble over

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

I have never wished to cater to the crowd; for what I know they do not
approve, and what they approve I do not know. -- Epicurus
-------------- 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/20080208/866f25dc/attachment.pgp>