Re: [libav-devel] [PATCH] vp8: Use 2 registers for dst_stride and src_stride in neon bilin filter.

2014-02-05 Thread Janne Grunau
On 2014-02-05 15:00:57 +0100, Christophe Gisquet wrote: > Hi, > > 2014-02-05 Martin Storsjö : > > Based on a patch by Ronald S. Bultje. > > --- > > Updated as suggested by Janne, to not require pushing anything to > > the stack. > > That's not really an issue worthy of additional work, but with s

Re: [libav-devel] [PATCH] vp8: Use 2 registers for dst_stride and src_stride in neon bilin filter.

2014-02-05 Thread Luca Barbato
On 05/02/14 22:59, Janne Grunau wrote: > On 2014-02-05 14:49:04 +0200, Martin Storsjö wrote: >> Based on a patch by Ronald S. Bultje. >> --- >> Updated as suggested by Janne, to not require pushing anything to >> the stack. >> --- >> libavcodec/arm/vp8dsp_neon.S | 120 >> +

Re: [libav-devel] [PATCH] vp8: Use 2 registers for dst_stride and src_stride in neon bilin filter.

2014-02-05 Thread Janne Grunau
On 2014-02-05 14:49:04 +0200, Martin Storsjö wrote: > Based on a patch by Ronald S. Bultje. > --- > Updated as suggested by Janne, to not require pushing anything to > the stack. > --- > libavcodec/arm/vp8dsp_neon.S | 120 > +-- > 1 file changed, 60 inserti

Re: [libav-devel] [PATCH] vp8: Use 2 registers for dst_stride and src_stride in neon bilin filter.

2014-02-05 Thread Martin Storsjö
On Wed, 5 Feb 2014, Christophe Gisquet wrote: Hi, 2014-02-05 Martin Storsjö : Based on a patch by Ronald S. Bultje. --- Updated as suggested by Janne, to not require pushing anything to the stack. That's not really an issue worthy of additional work, but with such patches, it may be interes

Re: [libav-devel] [PATCH] vp8: Use 2 registers for dst_stride and src_stride in neon bilin filter.

2014-02-05 Thread Christophe Gisquet
Hi, 2014-02-05 Martin Storsjö : > Based on a patch by Ronald S. Bultje. > --- > Updated as suggested by Janne, to not require pushing anything to > the stack. That's not really an issue worthy of additional work, but with such patches, it may be interesting to know the impact of the fix speed-wi

[libav-devel] [PATCH] vp8: Use 2 registers for dst_stride and src_stride in neon bilin filter.

2014-02-05 Thread Martin Storsjö
Based on a patch by Ronald S. Bultje. --- Updated as suggested by Janne, to not require pushing anything to the stack. --- libavcodec/arm/vp8dsp_neon.S | 120 +-- 1 file changed, 60 insertions(+), 60 deletions(-) diff --git a/libavcodec/arm/vp8dsp_neon.S b/

Re: [libav-devel] [PATCH] vp8: use 2 registers for dst_stride and src_stride in neon bilin filter.

2014-02-05 Thread Janne Grunau
On 2014-02-05 12:18:34 +0200, Martin Storsjö wrote: > From: "Ronald S. Bultje" > > --- > libavcodec/arm/vp8dsp_neon.S | 165 > +++ > 1 file changed, 87 insertions(+), 78 deletions(-) ok in the sense that the code does the right thing afterwards. It shoul

[libav-devel] [PATCH] vp8: use 2 registers for dst_stride and src_stride in neon bilin filter.

2014-02-05 Thread Martin Storsjö
From: "Ronald S. Bultje" --- libavcodec/arm/vp8dsp_neon.S | 165 +++ 1 file changed, 87 insertions(+), 78 deletions(-) diff --git a/libavcodec/arm/vp8dsp_neon.S b/libavcodec/arm/vp8dsp_neon.S index 0cd2efa..b36177e 100644 --- a/libavcodec/arm/vp8dsp_neon.