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

James Almer jamrial at gmail.com
Sat Jun 24 08:01:27 EEST 2017


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.

>      xorps       m0, m0
>      xorps       m1, m1
>      sar         r2, 3
> 


More information about the ffmpeg-devel mailing list