<div data-ntes="ntes_mail_body_root" style="line-height:1.7;color:#000000;font-size:14px;font-family:Arial"><div id="spnEditorContent"><p style="margin: 0;">Hi <span style="font-family: arial; white-space-collapse: preserve;">Micro,</span></p><p style="margin: 0;"><br></p><p style="margin: 0;">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.</p><pre style="width: 1238.04px; word-break: break-word !important;">+void pelFilterChroma_V_c(pixel *src, intptr_t srcStep, intptr_t offset, int32_t tc,
+                         int32_t maskP, int32_t maskQ)
+{
+    <b>assert</b>(offset == 1);
</pre><div><pre style="width: 1238.04px; word-break: break-word !important;">+void pelFilterChroma_H_c(pixel *src, intptr_t srcStep, intptr_t offset, int32_t tc,
+                         int32_t maskP, int32_t maskQ)
+{
+    <b>assert</b>(srcStep == 1);
</pre></div><div>Regards,</div><p style="margin: 0;">Chen</p><p style="margin: 0;"><br></p></div><div style="position:relative;zoom:1"></div><div id="divNeteaseMailCard"></div><p style="margin: 0;"><br></p><pre><br>At 2025-02-18 00:20:10, "Micro Daryl Robles" <microdaryl.robles@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@videolan.org
>https://mailman.videolan.org/listinfo/x265-devel
</pre></div>