[x265] [PATCH 1/5] AArch64: Add Neon implementation of pelFilterLumaStrong_V

[chen]

>+void pelFilterLumaStrong_V_neon(pixel *src, intptr_t srcStep, intptr_t offset,

>+                                int32_t tcP, int32_t tcQ)
>+{
>+    assert(offset == 1);
>+

>+    src -= offset * 4;
This function already assume offset=1, so we can remove it, replace with X265_CHECK




>+
>+    const int16x8_t tc_vec = vcombine_s16(vdup_n_s16(tcP), vdup_n_s16(tcQ));
>+    const int16x8_t neg_tc_vec = vnegq_s16(tc_vec);
> 

>+    const uint8_t filter[3][8] =
suggest prefix 'static const...', because some compiler (e.g. Microsoft Visual Studio) may generate really array initialize code other than constant memory.



-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20250212/a7824203/attachment.htm>