Re: [FFmpeg-devel] [PATCH/RFC] avutil/pixelutils: port ppc/altivec sad functions

2014-08-24 Thread Guillaume POIRIER
Hello, On Sun, Aug 24, 2014 at 9:39 PM, Clément Bœsch wrote: >> $ make fate-pixelutils >> TESTpixelutils >> --- ./tests/ref/fate/pixelutils 2014-08-24 12:18:09.0 +0200 >> +++ tests/data/fate/pixelutils 2014-08-24 14:20:02.0 +0200 >> @@ -1,7 +1,7 @@ >> [OK] [UU] SAD [random]

Re: [FFmpeg-devel] [PATCH/RFC] avutil/pixelutils: port ppc/altivec sad functions

2014-08-24 Thread Clément Bœsch
On Sun, Aug 24, 2014 at 12:21:38PM +, Carl Eugen Hoyos wrote: > Clément Bœsch pkh.me> writes: > > > +OBJS-$(CONFIG_PIXELUTILS) += ppc/pixelutils_init.o > > + > > ALTIVEC-OBJS += ppc/float_dsp_altivec.o \ > > +ALTIVEC-OBJS-$(CONFIG_PIXELUTILS) += ppc/pixelutils

Re: [FFmpeg-devel] [PATCH/RFC] avutil/pixelutils: port ppc/altivec sad functions

2014-08-24 Thread Carl Eugen Hoyos
Clément Bœsch pkh.me> writes: > +OBJS-$(CONFIG_PIXELUTILS) += ppc/pixelutils_init.o > + > ALTIVEC-OBJS += ppc/float_dsp_altivec.o \ > +ALTIVEC-OBJS-$(CONFIG_PIXELUTILS) += ppc/pixelutils_altivec.o \ Something is wrong here, I suspect the "\" have to be

Re: [FFmpeg-devel] [PATCH/RFC] avutil/pixelutils: port ppc/altivec sad functions

2014-08-24 Thread Clément Bœsch
On Sun, Aug 24, 2014 at 01:29:50PM +0200, Thilo Borgmann wrote: > Am 24.08.14 00:23, schrieb Michael Niedermayer: > > On Sat, Aug 23, 2014 at 09:24:33PM +0200, Clément Bœsch wrote: > >> --- This is 100% untested and probably doesn't even compile. > > > > it does compile, but i dont have a ppc so i

Re: [FFmpeg-devel] [PATCH/RFC] avutil/pixelutils: port ppc/altivec sad functions

2014-08-24 Thread Thilo Borgmann
Am 24.08.14 00:23, schrieb Michael Niedermayer: > On Sat, Aug 23, 2014 at 09:24:33PM +0200, Clément Bœsch wrote: >> --- This is 100% untested and probably doesn't even compile. > > it does compile, but i dont have a ppc so i cant say if it would work I can test that later today. 'make test' is w

Re: [FFmpeg-devel] [PATCH/RFC] avutil/pixelutils: port ppc/altivec sad functions

2014-08-23 Thread Michael Niedermayer
On Sat, Aug 23, 2014 at 09:24:33PM +0200, Clément Bœsch wrote: > --- > This is 100% untested and probably doesn't even compile. it does compile, but i dont have a ppc so i cant say if it would work [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB He who knows, do

[FFmpeg-devel] [PATCH/RFC] avutil/pixelutils: port ppc/altivec sad functions

2014-08-23 Thread Clément Bœsch
--- This is 100% untested and probably doesn't even compile. Can anyone with PPC/Altivec HW test or provide such access? After the altivec optims are ported we can drop the duplicated version in libavcodec entirely. The fate-pixelutils tests should cover the alignment checks; there might be some