[FFmpeg-devel] [PATCH 2/4] lavc/x86/hevc_add_res: Fix overflow in ADD_RES_SSE_8_8

Anton Khirnov anton at khirnov.net
Thu Mar 26 16:43:59 EET 2020


Quoting Linjie Fu (2020-03-05 08:47:54)
> Fix overflow for coeff -32768 in function ADD_RES_SSE_8_8 with
> no performance drop.
> 
> ./checkasm --test=hevc_add_res --bench
> 
> Mainline:
>   - hevc_add_res.add_residual [OK]
>     hevc_add_res_8x8_8_sse2: 15.5
> 
> Add overflow test case:
>   - hevc_add_res.add_residual [FAILED]
> 
> After:
>   - hevc_add_res.add_residual [OK]
>     hevc_add_res_8x8_8_sse2: 15.5
> 
> Signed-off-by: Xu Guangxin <guangxin.xu at intel.com>
> Signed-off-by: Linjie Fu <linjie.fu at intel.com>
> ---
>  libavcodec/x86/hevc_add_res.asm | 45 ++++++++++++++++++++---------------------
>  1 file changed, 22 insertions(+), 23 deletions(-)

This and 3/4 acked by Lynne on IRC, so will push later if nobody
objects.

-- 
Anton Khirnov


More information about the ffmpeg-devel mailing list