Re: [FFmpeg-devel] [PATCH] swresample/arm: add ff_resample_common_apply_filter_{x4, x8}_{float, s16}_neon

2016-05-13 Thread Matthieu Bouron
On Thu, May 12, 2016 at 3:50 PM, Benoit Fouet wrote: > Hi, > > > On 12/05/2016 15:22, Matthieu Bouron wrote: > >> On Thu, May 12, 2016 at 10:01 AM, Benoit Fouet >> wrote: >> >> Hi, >>> >>> I mostly have nits remarks. >>> >>> On 11/05/2016 18:39,

Re: [FFmpeg-devel] [PATCH] swresample/arm: add ff_resample_common_apply_filter_{x4, x8}_{float, s16}_neon

2016-05-12 Thread Benoit Fouet
Hi, On 12/05/2016 15:22, Matthieu Bouron wrote: On Thu, May 12, 2016 at 10:01 AM, Benoit Fouet wrote: Hi, I mostly have nits remarks. On 11/05/2016 18:39, Matthieu Bouron wrote: From: Matthieu Bouron [...] diff --git

Re: [FFmpeg-devel] [PATCH] swresample/arm: add ff_resample_common_apply_filter_{x4, x8}_{float, s16}_neon

2016-05-12 Thread Matthieu Bouron
On Thu, May 12, 2016 at 10:01 AM, Benoit Fouet wrote: > Hi, > > I mostly have nits remarks. > > On 11/05/2016 18:39, Matthieu Bouron wrote: > >> From: Matthieu Bouron >> >> > [...] > > diff --git a/libswresample/arm/resample.S

Re: [FFmpeg-devel] [PATCH] swresample/arm: add ff_resample_common_apply_filter_{x4, x8}_{float, s16}_neon

2016-05-12 Thread Benoit Fouet
Hi, I mostly have nits remarks. On 11/05/2016 18:39, Matthieu Bouron wrote: From: Matthieu Bouron [...] diff --git a/libswresample/arm/resample.S b/libswresample/arm/resample.S new file mode 100644 index 000..13462e3 --- /dev/null +++

Re: [FFmpeg-devel] [PATCH] swresample/arm: add ff_resample_common_apply_filter_{x4, x8}_{float, s16}_neon

2016-05-11 Thread Michael Niedermayer
On Wed, May 11, 2016 at 10:02:33PM +0200, Matthieu Bouron wrote: > On Wed, May 11, 2016 at 9:04 PM, Reimar Döffinger > wrote: > > > > > > > On 11.05.2016, at 20:37, Michael Niedermayer > > wrote: > > > > > On Wed, May 11, 2016 at 06:39:20PM

Re: [FFmpeg-devel] [PATCH] swresample/arm: add ff_resample_common_apply_filter_{x4, x8}_{float, s16}_neon

2016-05-11 Thread Matthieu Bouron
Le 11 mai 2016 6:39 PM, "Matthieu Bouron" a écrit : > > From: Matthieu Bouron > > --- > > Hello, > > Here are some benchmark on a rpi2 of the attached patch. > > ./ffmpeg -f lavfi -i

Re: [FFmpeg-devel] [PATCH] swresample/arm: add ff_resample_common_apply_filter_{x4, x8}_{float, s16}_neon

2016-05-11 Thread Matthieu Bouron
On Wed, May 11, 2016 at 9:04 PM, Reimar Döffinger wrote: > > > On 11.05.2016, at 20:37, Michael Niedermayer > wrote: > > > On Wed, May 11, 2016 at 06:39:20PM +0200, Matthieu Bouron wrote: > >> From: Matthieu Bouron

Re: [FFmpeg-devel] [PATCH] swresample/arm: add ff_resample_common_apply_filter_{x4, x8}_{float, s16}_neon

2016-05-11 Thread Reimar Döffinger
On 11.05.2016, at 21:09, James Almer wrote: > On 5/11/2016 4:04 PM, Reimar Döffinger wrote: >> otherwise there's a risk that enabling the recently discussed >> -ftree-vecorize for that file specifically would give better results. > > Tree vectorize is enabled for x86 and GCC

Re: [FFmpeg-devel] [PATCH] swresample/arm: add ff_resample_common_apply_filter_{x4, x8}_{float, s16}_neon

2016-05-11 Thread James Almer
On 5/11/2016 4:04 PM, Reimar Döffinger wrote: > otherwise there's a risk that enabling the recently discussed -ftree-vecorize > for that file specifically would give better results. Tree vectorize is enabled for x86 and GCC >= 4.9 only. And even then it may not even remain like that much longer.

Re: [FFmpeg-devel] [PATCH] swresample/arm: add ff_resample_common_apply_filter_{x4, x8}_{float, s16}_neon

2016-05-11 Thread Reimar Döffinger
On 11.05.2016, at 20:37, Michael Niedermayer wrote: > On Wed, May 11, 2016 at 06:39:20PM +0200, Matthieu Bouron wrote: >> From: Matthieu Bouron >> >> --- >> >> Hello, >> >> Here are some benchmark on a rpi2 of the attached patch. >>

Re: [FFmpeg-devel] [PATCH] swresample/arm: add ff_resample_common_apply_filter_{x4, x8}_{float, s16}_neon

2016-05-11 Thread Michael Niedermayer
On Wed, May 11, 2016 at 06:39:20PM +0200, Matthieu Bouron wrote: > From: Matthieu Bouron > > --- > > Hello, > > Here are some benchmark on a rpi2 of the attached patch. > > ./ffmpeg -f lavfi -i >

[FFmpeg-devel] [PATCH] swresample/arm: add ff_resample_common_apply_filter_{x4, x8}_{float, s16}_neon

2016-05-11 Thread Matthieu Bouron
From: Matthieu Bouron --- Hello, Here are some benchmark on a rpi2 of the attached patch. ./ffmpeg -f lavfi -i sine=440,aformat=sample_fmts=fltp,asetnsamples=4096,abench=start,aresample=48000,abench=stop -t 1000 -f null - With patch:avg=0.001159