[x265] [PATCH v2 0/5] AArch64: Add Neon/Unroll impl of pelFilters

[chen]

Hi Micro,




Thank for the revised patches, the patch 1/5 ~ 3/5 looks good to me, the 4/5 and 5/5 lack some X265_CHECK, other are fine.

+void pelFilterChroma_V_c(pixel *src, intptr_t srcStep, intptr_t offset, int32_t tc,
+                         int32_t maskP, int32_t maskQ)
+{
+    assert(offset == 1);

+void pelFilterChroma_H_c(pixel *src, intptr_t srcStep, intptr_t offset, int32_t tc,
+                         int32_t maskP, int32_t maskQ)
+{
+    assert(srcStep == 1);

Regards,

Chen








At 2025-02-18 00:20:10, "Micro Daryl Robles" <microdaryl.robles at arm.com> wrote:
>Hi Chen,
>
>Thank you for your comments. I am submitting this v2 patch to include
>your suggestions.
>
>Many thanks,
>Micro
>
>Micro Daryl Robles (5):
>  AArch64: Add Neon implementation of pelFilterLumaStrong_V
>  AArch64: Add Neon implementation of pelFilterLumaStrong_H
>  Change the test conditions for pelFilterChroma in PixelHarness
>  Unroll C implementation of pelFilterChroma_V
>  Unroll C implementation of pelFilterChroma_H
>
> source/common/aarch64/loopfilter-prim.cpp | 134 +++++++++++++++++++++-
> source/common/loopfilter.cpp              | 106 +++++++++++++----
> source/test/pixelharness.cpp              | 109 +++++++++++-------
> 3 files changed, 286 insertions(+), 63 deletions(-)
>
>-- 
>2.34.1
>
>_______________________________________________
>x265-devel mailing list
>x265-devel at videolan.org
>https://mailman.videolan.org/listinfo/x265-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20250218/3f435ba4/attachment-0001.htm>