[FFmpeg-devel] libavutil simd

Diego Biurrun diego
Tue Oct 2 13:40:03 CEST 2007


On Tue, Oct 02, 2007 at 01:19:12PM +0200, Luca Barbato wrote:
> Diego Biurrun wrote:
> > On Tue, Oct 02, 2007 at 12:36:21PM +0200, Luca Barbato wrote:
> >> here the patch, it works fine for ppc, Diego have a look at the Makefile
> >> changes.
> >>
> >> check_altivec.c MUST BE BUILT w/out -maltivec
> > 
> > That can be achieved without such a lot of Makefile hackery, for example
> > with something like (untested):
> > 
> > ppc/check_altivec.o: CFLAGS := $(filter-out -maltivec,$(CFLAGS))
> 
> The rest must have -maltivec -mabi=altivec on...

Yes, but you still have the logic backwards, that's why your patch is so
intrusive.  Just turn it off for the one file that does not need it.

> >> --- ppc/imgresample_altivec.h	(revision 0)
> >> +++ ppc/imgresample_altivec.h	(revision 0)
> >> @@ -0,0 +1,26 @@
> >> +/*
> >> + * Copyright (c) 2001 Fabrice Bellard. 
> > 
> > trailing whitespace
> > 
> > Also, is Fabrice the author of this code?  I don't think so, just remove
> > the line.
> 
> The unsplitted source header got preserved...

The source header is not a copyrightable work.

Diego




More information about the ffmpeg-devel mailing list