[FFmpeg-devel] [PATCH v2 4/6] lavc/aarch64: add hevc sao edge 8x8

Martin Storsjö martin at martin.st
Thu Nov 18 10:47:52 EET 2021


On Wed, 17 Nov 2021, J. Dekker wrote:

> --bench on AWS Graviton:
>
> hevc_sao_edge_8x8_8_c: 516.0
> hevc_sao_edge_8x8_8_neon: 81.0
>
> Signed-off-by: J. Dekker <jdek at itanimul.li>
> ---
> libavcodec/aarch64/hevcdsp_init_aarch64.c |  3 ++
> libavcodec/aarch64/hevcdsp_sao_neon.S     | 51 +++++++++++++++++++++++
> 2 files changed, 54 insertions(+)
>
> Used inline shift for ldr & escheduled uxtl/uxtl2 before zip/zip2.

Code wise ok, but please squash the reindentation of this particular 
function from patch 6/6 into this patch; there's no point in first 
committing one version and fixing the indentation in the next patch.

// Martin



More information about the ffmpeg-devel mailing list