[FFmpeg-devel] [PATCH] fix compiling VIS assembly on SPARC

[Diego Biurrun]

On Wed, Sep 12, 2007 at 11:28:01AM +0200, Diego Biurrun wrote:
> On Tue, Sep 11, 2007 at 11:53:54PM +0200, Balatoni Denes wrote:
> > 
> > So this was missed before by me. When building 
> > simple_idct_vis -mcpu=ultrasparc has to be given to gcc. I hope the patch is 
> > okay, because I am not a makefile guru.
> > 
> > --- ffmpeg.old/libavcodec/Makefile	2007-08-30 23:37:54.000000000 +0200
> > +++ ffmpeg/libavcodec/Makefile	2007-09-11 10:50:43.000000000 +0200
> > @@ -380,6 +380,8 @@
> >  
> >  sparc/dsputil_vis.o: CFLAGS += -mcpu=ultrasparc -mtune=ultrasparc
> >  
> > +sparc/simple_idct_vis.o: CFLAGS += -mcpu=ultrasparc -mtune=ultrasparc
> 
> You can merge the two rules like this:
> 
>   sparc/dsputil_vis.o sparc/simple_idct_vis.o: CFLAGS += -mcpu=ultrasparc -mtune=ultrasparc

Applied with that modification.

> But I wonder why this has to be set only for those two files and not for
> everything ...

Denes, could you tell me why this is not set by configure, possibly for
all files?

Diego