[FFmpeg-devel] [PATCH] lavc/x86: clear r2 higher bits in ff_sbr_sum_square

Matthieu Bouron matthieu.bouron at gmail.com
Mon Jun 26 10:59:05 EEST 2017


On Sat, Jun 24, 2017 at 02:01:27AM -0300, James Almer wrote:
> On 6/23/2017 11:56 AM, Matthieu Bouron wrote:
> > Suggested-by: James Almer <jamrial at gmail.com>
> > ---
> >  libavcodec/x86/sbrdsp.asm | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/libavcodec/x86/sbrdsp.asm b/libavcodec/x86/sbrdsp.asm
> > index 07a412b2ae..02b52ee300 100644
> > --- a/libavcodec/x86/sbrdsp.asm
> > +++ b/libavcodec/x86/sbrdsp.asm
> > @@ -38,7 +38,7 @@ SECTION .text
> >  
> >  INIT_XMM sse
> >  cglobal sbr_sum_square, 2, 3, 6
> > -    mov         r2, r1
> > +    mov         r2d, r1d
> 
> nit: remove one space after mov to get vertical alignment.
> 
> LGTM either way, of course.

Applied with fixed alignment.

Thanks.

[...]

-- 
Matthieu B.


More information about the ffmpeg-devel mailing list