Hi Gerda,



The patches looks good, thank you.




Just a little comment, unncessary modify patch.

in the code 'in + 0' --> &in[0], it help clear indicates that is an access to a 
local array.




Regards,

Chen

At 2025-02-22 00:04:39, "Gerda Zsejke More" <[email protected]> wrote:
>Hi,
>
>This patch series adds Neon intrinsic optimisations of interp functions.
>
>Many thanks,
>Gerda
>
>Gerda Zsejke More (10):
>  AArch64: Optimise HBD interp_horiz_pp_neon
>  AArch64: Optimise HBD interp_horiz_pp_neon for 10-bit
>  AArch64: Optimise HBD interp_horiz_ps_neon
>  AArch64: Optimise HBD interp_horiz_ps_neon for 10-bit
>  AArch64: Optimise HBD interp_vert_pp_neon
>  AArch64: Optimise HBD interp_vert_ps_neon
>  AArch64: Optimise SBD interp_vert_ss/sp
>  AArch64: Optimise HBD interp_vert_sp_neon
>  AArch64: Enable existing interp_vert_ss impl for HBD
>  AArch64: Clean up interp function definitions
>
> source/common/aarch64/filter-prim.cpp | 3561 ++++++++++++++++++++-----
> source/common/aarch64/mem-neon.h      |  154 +-
> 2 files changed, 2992 insertions(+), 723 deletions(-)
>
>-- 
>2.39.5 (Apple Git-154)
>
>_______________________________________________
>x265-devel mailing list
>[email protected]
>https://mailman.videolan.org/listinfo/x265-devel
_______________________________________________
x265-devel mailing list
[email protected]
https://mailman.videolan.org/listinfo/x265-devel

Reply via email to