[FFmpeg-devel] flac_compute_autocorr_sse2 not pic-safe

[Edward Hervey]

non-ubuntu-fucked-up version of the last lines of compilation, I
apologize for that.

i386/dsputil_mmx.c:3762: warning: assignment from incompatible pointer
type
i386/dsputil_mmx.c:3770: warning: assignment from incompatible pointer
type
i386/dsputil_mmx.c: In function 'flac_compute_autocorr_sse2':
i386/dsputil_mmx.c:3039: error: can't find a register in class
'GENERAL_REGS' while reloading 'asm'
make[1]: *** [i386/dsputil_mmx.o] Error 1
make[1]: Leaving directory `/home/bilboed/work/cvs/ffmpeg/libavcodec'
make: *** [lib] Error 2


On Tue, 2007-12-04 at 19:50 +0100, Michael Niedermayer wrote:
> On Tue, Dec 04, 2007 at 07:35:31PM +0100, Luca Barbato wrote:
> > Michael Niedermayer wrote:
> > > On Tue, Dec 04, 2007 at 03:27:18PM +0100, Edward Hervey wrote:
> > >> Hi all,
> > >>
> > >>   While compiling ffmpeg as static library to be put into a shared
> > >> library (./configure --enable-static --disable-shared
> > >> --extra-cflags=-fPIC), I encountered a piece of asm code which isn't
> > >> PIC-safe.
> > >>
> > >>   The proposed patch disables that optimized routine when the
> > >> configuration doesn't allow it (HAVE_7REGS set to 1).
> > > 
> > > patch rejected, this code does not need 7 registers
> > 
> >  :"+&r"(i), "=m"(autoc[j]), "=m"(autoc[j+1]), "=m"(autoc[j+2])
> >  :"r"(data1+len), "r"(data1+len-j), "m"(*ff_pd_1)
> > 
> > gcc being overzealous?
> > what about marking this a BROKEN_RELOCATIONS ?
> 
> do you know more then the rest of us?
> nothing in the mail suggests a relation to relocations
> 
> but then its not a proper bug report, we dont know the CFLAGS nor the exact
> error message nor if this is x86_64 so maybe it is related to relocations
> 
> [...]
> 
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at mplayerhq.hu
> http://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-devel