[FFmpeg-devel] [PATCH] lavc/aarch64/simple_idct: fix idct_col4_top coefficient

Matthieu Bouron matthieu.bouron at gmail.com
Tue Jun 13 18:50:37 EEST 2017


On Tue, Jun 13, 2017 at 05:26:43PM +0200, Clément Bœsch wrote:
> On Tue, Jun 13, 2017 at 05:24:21PM +0200, Matthieu Bouron wrote:
> > Fixes regression introduced by 5d0b8b1ae307951310c7d9a8fa282fbca9b997cd.
> > ---
> >  libavcodec/aarch64/simple_idct_neon.S | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/libavcodec/aarch64/simple_idct_neon.S b/libavcodec/aarch64/simple_idct_neon.S
> > index 92987985d2..5bd31e5be9 100644
> > --- a/libavcodec/aarch64/simple_idct_neon.S
> > +++ b/libavcodec/aarch64/simple_idct_neon.S
> > @@ -74,7 +74,7 @@ endconst
> >  .endm
> >  
> >  .macro idct_col4_top y1, y2, y3, y4, i, l
> > -        smull\i         v7.4S,  \y3\l, z1
> > +        smull\i         v7.4S,  \y3\l, z2
> >          smull\i         v16.4S, \y3\l, z6
> >          smull\i         v17.4S, \y2\l, z1
> >          add             v19.4S, v23.4S, v7.4S
> 
> LGTM

Patch applied and backported to the 3.3 branch.

Thanks.

-- 
Matthieu B.


More information about the ffmpeg-devel mailing list