[x265] [PATCH 6/7] AArch64: Add SVE saoCuStats primitives

Hari Limaye hari.limaye at arm.com
Mon May 20 16:18:44 UTC 2024


Add optimised implementations of saoCuStats primitives using SVE
instructions for both low and high bitdepth.

Results of speedtests, compared to the existing C and Neon code, when
compiled with LLVM 17 on a Neoverse V1 machine (higher is better):

Low bitdepth:

                 | C -> SVE | Neon -> SVE |
    saoCuStatsE0 |    2.97x |       1.12x |
    saoCuStatsE1 |    3.46x |       1.15x |
    saoCuStatsE2 |    3.49x |       1.21x |
    saoCuStatsE3 |    3.72x |       1.14x |

High bitdepth:

                 | C -> SVE | Neon -> SVE |
    saoCuStatsE0 |    2.84x |       1.19x |
    saoCuStatsE1 |    3.47x |       1.28x |
    saoCuStatsE2 |    3.01x |       1.19x |
    saoCuStatsE3 |    3.44x |       1.12x |
---
 source/common/CMakeLists.txt             |  12 +-
 source/common/aarch64/asm-primitives.cpp |   6 +
 source/common/aarch64/sao-prim-sve.cpp   | 298 +++++++++++++++++++++++
 source/common/aarch64/sao-prim.cpp       |  27 --
 source/common/aarch64/sao-prim.h         |  32 +++
 5 files changed, 346 insertions(+), 29 deletions(-)
 create mode 100644 source/common/aarch64/sao-prim-sve.cpp

diff --git a/source/common/CMakeLists.txt b/source/common/CMakeLists.txt
index 1d7cd3cd0..40c932966 100644
--- a/source/common/CMakeLists.txt
+++ b/source/common/CMakeLists.txt
@@ -103,7 +103,8 @@ if(ENABLE_ASSEMBLY AND (ARM64 OR CROSS_COMPILE_ARM64))
         add_definitions(-DAUTO_VECTORIZE=1)
     endif()
 
-    set(C_SRCS asm-primitives.cpp pixel-prim.h pixel-prim.cpp filter-prim.h filter-prim.cpp dct-prim.h dct-prim.cpp loopfilter-prim.cpp loopfilter-prim.h intrapred-prim.cpp arm64-utils.cpp arm64-utils.h fun-decls.h sao-prim.cpp)
+    set(C_SRCS_NEON asm-primitives.cpp pixel-prim.h pixel-prim.cpp filter-prim.h filter-prim.cpp dct-prim.h dct-prim.cpp loopfilter-prim.cpp loopfilter-prim.h intrapred-prim.cpp arm64-utils.cpp arm64-utils.h fun-decls.h sao-prim.cpp)
+    set(C_SRCS_SVE sao-prim-sve.cpp)
     enable_language(ASM)
 
     # add ARM assembly/intrinsic files here
@@ -115,9 +116,16 @@ if(ENABLE_ASSEMBLY AND (ARM64 OR CROSS_COMPILE_ARM64))
     set(ARM_ASMS "${A_SRCS}" CACHE INTERNAL "ARM Assembly Sources")
     set(ARM_ASMS_SVE "${A_SRCS_SVE}" CACHE INTERNAL "ARM Assembly Sources that use SVE instruction set")
     set(ARM_ASMS_SVE2 "${A_SRCS_SVE2}" CACHE INTERNAL "ARM Assembly Sources that use SVE2 instruction set")
-    foreach(SRC ${C_SRCS})
+    foreach(SRC ${C_SRCS_NEON})
         set(ASM_PRIMITIVES ${ASM_PRIMITIVES} aarch64/${SRC})
     endforeach()
+
+    if(CPU_HAS_SVE AND HAVE_SVE_BRIDGE)
+        foreach(SRC ${C_SRCS_SVE})
+            set(ASM_PRIMITIVES ${ASM_PRIMITIVES} aarch64/${SRC})
+        endforeach()
+    endif()
+
     source_group(Assembly FILES ${ASM_PRIMITIVES})
 endif(ENABLE_ASSEMBLY AND (ARM64 OR CROSS_COMPILE_ARM64))
 
diff --git a/source/common/aarch64/asm-primitives.cpp b/source/common/aarch64/asm-primitives.cpp
index afcab7ad7..bab34a493 100644
--- a/source/common/aarch64/asm-primitives.cpp
+++ b/source/common/aarch64/asm-primitives.cpp
@@ -1952,6 +1952,12 @@ void setupIntrinsicPrimitives(EncoderPrimitives &p, int cpuMask)
         setupIntraPrimitives_neon(p);
         setupSaoPrimitives_neon(p);
     }
+#if defined(HAVE_SVE) && HAVE_SVE_BRIDGE
+    if (cpuMask & X265_CPU_SVE)
+    {
+        setupSaoPrimitives_sve(p);
+    }
+#endif
 }
 
 } // namespace X265_NS
diff --git a/source/common/aarch64/sao-prim-sve.cpp b/source/common/aarch64/sao-prim-sve.cpp
new file mode 100644
index 000000000..4b9e3c5d2
--- /dev/null
+++ b/source/common/aarch64/sao-prim-sve.cpp
@@ -0,0 +1,298 @@
+/*****************************************************************************
+ * Copyright (C) 2024 MulticoreWare, Inc
+ *
+ * Authors: Hari Limaye <hari.limaye at arm.com>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02111, USA.
+ *
+ * This program is also available under a commercial proprietary license.
+ * For more information, contact us at license @ x265.com.
+ *****************************************************************************/
+
+#include "sao-prim.h"
+#include <arm_neon_sve_bridge.h>
+
+/* We can access instructions that are exclusive to the SVE instruction set from
+ * a predominantly Neon context by making use of the Neon-SVE bridge intrinsics
+ * to reinterpret Neon vectors as SVE vectors - with the high part of the SVE
+ * vector (if it's longer than 128 bits) being "don't care".
+ *
+ * While sub-optimal on machines that have SVE vector length > 128-bit - as the
+ * remainder of the vector is unused - this approach is still beneficial when
+ * compared to a Neon-only implementation. */
+
+static inline int8x16_t x265_sve_mask(const int x, const int endX,
+                                      const int8x16_t in)
+{
+    // Use predicate to shift "unused lanes" outside of range [-2, 2]
+    svbool_t svpred = svwhilelt_b8(x, endX);
+    svint8_t edge_type = svsel_s8(svpred, svset_neonq_s8(svundef_s8(), in),
+                                  svdup_n_s8(-3));
+    return svget_neonq_s8(edge_type);
+}
+
+static inline int64x2_t x265_sdotq_s16(int64x2_t acc, int16x8_t x, int16x8_t y)
+{
+    return svget_neonq_s64(svdot_s64(svset_neonq_s64(svundef_s64(), acc),
+                                     svset_neonq_s16(svundef_s16(), x),
+                                     svset_neonq_s16(svundef_s16(), y)));
+}
+
+/*
+ * Compute Edge Offset statistics (count and stats).
+ * To save some instructions compute count and stats as negative values - since
+ * output of Neon comparison instructions for a matched condition is all 1s (-1).
+ */
+static inline void compute_eo_stats(const int8x16_t edge_type,
+                                    const int16_t *diff, int16x8_t *count,
+                                    int64x2_t *stats)
+{
+    // Create a mask for each edge type.
+    int8x16_t mask0 = vreinterpretq_s8_u8(vceqq_s8(edge_type, vdupq_n_s8(-2)));
+    int8x16_t mask1 = vreinterpretq_s8_u8(vceqq_s8(edge_type, vdupq_n_s8(-1)));
+    int8x16_t mask2 = vreinterpretq_s8_u8(vceqq_s8(edge_type, vdupq_n_s8(0)));
+    int8x16_t mask3 = vreinterpretq_s8_u8(vceqq_s8(edge_type, vdupq_n_s8(1)));
+    int8x16_t mask4 = vreinterpretq_s8_u8(vceqq_s8(edge_type, vdupq_n_s8(2)));
+
+    // Compute negative counts for each edge type.
+    count[0] = vpadalq_s8(count[0], mask0);
+    count[1] = vpadalq_s8(count[1], mask1);
+    count[2] = vpadalq_s8(count[2], mask2);
+    count[3] = vpadalq_s8(count[3], mask3);
+    count[4] = vpadalq_s8(count[4], mask4);
+
+    // Widen the masks to 16-bit.
+    int16x8_t mask0_lo = vreinterpretq_s16_s8(vzip1q_s8(mask0, mask0));
+    int16x8_t mask0_hi = vreinterpretq_s16_s8(vzip2q_s8(mask0, mask0));
+    int16x8_t mask1_lo = vreinterpretq_s16_s8(vzip1q_s8(mask1, mask1));
+    int16x8_t mask1_hi = vreinterpretq_s16_s8(vzip2q_s8(mask1, mask1));
+    int16x8_t mask2_lo = vreinterpretq_s16_s8(vzip1q_s8(mask2, mask2));
+    int16x8_t mask2_hi = vreinterpretq_s16_s8(vzip2q_s8(mask2, mask2));
+    int16x8_t mask3_lo = vreinterpretq_s16_s8(vzip1q_s8(mask3, mask3));
+    int16x8_t mask3_hi = vreinterpretq_s16_s8(vzip2q_s8(mask3, mask3));
+    int16x8_t mask4_lo = vreinterpretq_s16_s8(vzip1q_s8(mask4, mask4));
+    int16x8_t mask4_hi = vreinterpretq_s16_s8(vzip2q_s8(mask4, mask4));
+
+    int16x8_t diff_lo = vld1q_s16(diff);
+    int16x8_t diff_hi = vld1q_s16(diff + 8);
+
+    // Compute negative stats for each edge type.
+    stats[0] = x265_sdotq_s16(stats[0], diff_lo, mask0_lo);
+    stats[0] = x265_sdotq_s16(stats[0], diff_hi, mask0_hi);
+    stats[1] = x265_sdotq_s16(stats[1], diff_lo, mask1_lo);
+    stats[1] = x265_sdotq_s16(stats[1], diff_hi, mask1_hi);
+    stats[2] = x265_sdotq_s16(stats[2], diff_lo, mask2_lo);
+    stats[2] = x265_sdotq_s16(stats[2], diff_hi, mask2_hi);
+    stats[3] = x265_sdotq_s16(stats[3], diff_lo, mask3_lo);
+    stats[3] = x265_sdotq_s16(stats[3], diff_hi, mask3_hi);
+    stats[4] = x265_sdotq_s16(stats[4], diff_lo, mask4_lo);
+    stats[4] = x265_sdotq_s16(stats[4], diff_hi, mask4_hi);
+}
+
+/*
+ * Reduce and store Edge Offset statistics (count and stats).
+ */
+static inline void reduce_eo_stats(int64x2_t *vstats, int16x8_t *vcount,
+                                   int32_t *stats, int32_t *count)
+{
+    // s_eoTable maps edge types to memory in order: {2, 0, 1, 3, 4}.
+    int16x8_t c01 = vpaddq_s16(vcount[2], vcount[0]);
+    int16x8_t c23 = vpaddq_s16(vcount[1], vcount[3]);
+    int16x8_t c0123 = vpaddq_s16(c01, c23);
+    // Subtract from current count, as we calculate the negation.
+    vst1q_s32(count, vsubq_s32(vld1q_s32(count), vpaddlq_s16(c0123)));
+    count[4] -= vaddvq_s16(vcount[4]);
+
+    int32x4_t s01 = vcombine_s32(vmovn_s64(vstats[2]), vmovn_s64(vstats[0]));
+    int32x4_t s23 = vcombine_s32(vmovn_s64(vstats[1]), vmovn_s64(vstats[3]));
+    int32x4_t s0123 = vpaddq_s32(s01, s23);
+    // Subtract from current stats, as we calculate the negation.
+    vst1q_s32(stats, vsubq_s32(vld1q_s32(stats), s0123));
+    stats[4] -= vaddvq_s64(vstats[4]);
+}
+
+namespace X265_NS {
+void saoCuStatsE0_sve(const int16_t *diff, const pixel *rec, intptr_t stride,
+                      int endX, int endY, int32_t *stats, int32_t *count)
+{
+    // Separate buffers for each edge type, so that we can vectorise.
+    int16x8_t tmp_count[5] = { vdupq_n_s16(0), vdupq_n_s16(0), vdupq_n_s16(0),
+                               vdupq_n_s16(0), vdupq_n_s16(0) };
+    int64x2_t tmp_stats[5] = { vdupq_n_s64(0), vdupq_n_s64(0), vdupq_n_s64(0),
+                               vdupq_n_s64(0), vdupq_n_s64(0) };
+
+    for (int y = 0; y < endY; y++)
+    {
+        // Calculate negated sign_left(x) directly, to save negation when
+        // reusing sign_right(x) as sign_left(x + 1).
+        int8x16_t neg_sign_left = vdupq_n_s8(x265_signOf(rec[-1] - rec[0]));
+        for (int x = 0; x < endX; x += 16)
+        {
+            int8x16_t sign_right = signOf_neon(rec + x, rec + x + 1);
+
+            // neg_sign_left(x) = sign_right(x + 1), reusing one from previous
+            // iteration.
+            neg_sign_left = vextq_s8(neg_sign_left, sign_right, 15);
+
+            // Subtract instead of add, as sign_left is negated.
+            int8x16_t edge_type = vsubq_s8(sign_right, neg_sign_left);
+
+            // For reuse in the next iteration.
+            neg_sign_left = sign_right;
+
+            edge_type = x265_sve_mask(x, endX, edge_type);
+            compute_eo_stats(edge_type, diff + x, tmp_count, tmp_stats);
+        }
+
+        diff += MAX_CU_SIZE;
+        rec += stride;
+    }
+
+    reduce_eo_stats(tmp_stats, tmp_count, stats, count);
+}
+
+void saoCuStatsE1_sve(const int16_t *diff, const pixel *rec, intptr_t stride,
+                      int8_t *upBuff1, int endX, int endY, int32_t *stats,
+                      int32_t *count)
+{
+    // Separate buffers for each edge type, so that we can vectorise.
+    int16x8_t tmp_count[5] = { vdupq_n_s16(0), vdupq_n_s16(0), vdupq_n_s16(0),
+                               vdupq_n_s16(0), vdupq_n_s16(0) };
+    int64x2_t tmp_stats[5] = { vdupq_n_s64(0), vdupq_n_s64(0), vdupq_n_s64(0),
+                               vdupq_n_s64(0), vdupq_n_s64(0) };
+
+    // Negate upBuff1 (sign_up), so we can subtract and save repeated negations.
+    for (int x = 0; x < endX; x += 16)
+    {
+        vst1q_s8(upBuff1 + x, vnegq_s8(vld1q_s8(upBuff1 + x)));
+    }
+
+    for (int y = 0; y < endY; y++)
+    {
+        for (int x = 0; x < endX; x += 16)
+        {
+            int8x16_t sign_up = vld1q_s8(upBuff1 + x);
+            int8x16_t sign_down = signOf_neon(rec + x, rec + x + stride);
+
+            // Subtract instead of add, as sign_up is negated.
+            int8x16_t edge_type = vsubq_s8(sign_down, sign_up);
+
+            // For reuse in the next iteration.
+            vst1q_s8(upBuff1 + x, sign_down);
+
+            edge_type = x265_sve_mask(x, endX, edge_type);
+            compute_eo_stats(edge_type, diff + x, tmp_count, tmp_stats);
+        }
+
+        diff += MAX_CU_SIZE;
+        rec += stride;
+    }
+
+    reduce_eo_stats(tmp_stats, tmp_count, stats, count);
+}
+
+void saoCuStatsE2_sve(const int16_t *diff, const pixel *rec, intptr_t stride,
+                      int8_t *upBuff1, int8_t *upBufft, int endX, int endY,
+                      int32_t *stats, int32_t *count)
+{
+    // Separate buffers for each edge type, so that we can vectorise.
+    int16x8_t tmp_count[5] = { vdupq_n_s16(0), vdupq_n_s16(0), vdupq_n_s16(0),
+                               vdupq_n_s16(0), vdupq_n_s16(0) };
+    int64x2_t tmp_stats[5] = { vdupq_n_s64(0), vdupq_n_s64(0), vdupq_n_s64(0),
+                               vdupq_n_s64(0), vdupq_n_s64(0) };
+
+    // Negate upBuff1 (sign_up) so we can subtract and save repeated negations.
+    for (int x = 0; x < endX; x += 16)
+    {
+        vst1q_s8(upBuff1 + x, vnegq_s8(vld1q_s8(upBuff1 + x)));
+    }
+
+    for (int y = 0; y < endY; y++)
+    {
+        upBufft[0] = x265_signOf(rec[-1] - rec[stride]);
+        for (int x = 0; x < endX; x += 16)
+        {
+            int8x16_t sign_up = vld1q_s8(upBuff1 + x);
+            int8x16_t sign_down = signOf_neon(rec + x, rec + x + stride + 1);
+
+            // Subtract instead of add, as sign_up is negated.
+            int8x16_t edge_type = vsubq_s8(sign_down, sign_up);
+
+            // For reuse in the next iteration.
+            vst1q_s8(upBufft + x + 1, sign_down);
+
+            edge_type = x265_sve_mask(x, endX, edge_type);
+            compute_eo_stats(edge_type, diff + x, tmp_count, tmp_stats);
+        }
+
+        std::swap(upBuff1, upBufft);
+
+        rec += stride;
+        diff += MAX_CU_SIZE;
+    }
+
+    reduce_eo_stats(tmp_stats, tmp_count, stats, count);
+}
+
+void saoCuStatsE3_sve(const int16_t *diff, const pixel *rec, intptr_t stride,
+                      int8_t *upBuff1, int endX, int endY, int32_t *stats,
+                      int32_t *count)
+{
+    // Separate buffers for each edge type, so that we can vectorise.
+    int16x8_t tmp_count[5] = { vdupq_n_s16(0), vdupq_n_s16(0), vdupq_n_s16(0),
+                               vdupq_n_s16(0), vdupq_n_s16(0) };
+    int64x2_t tmp_stats[5] = { vdupq_n_s64(0), vdupq_n_s64(0), vdupq_n_s64(0),
+                               vdupq_n_s64(0), vdupq_n_s64(0) };
+
+    // Negate upBuff1 (sign_up) so we can subtract and save repeated negations.
+    for (int x = 0; x < endX; x += 16)
+    {
+        vst1q_s8(upBuff1 + x, vnegq_s8(vld1q_s8(upBuff1 + x)));
+    }
+
+    for (int y = 0; y < endY; y++)
+    {
+        for (int x = 0; x < endX; x += 16)
+        {
+            int8x16_t sign_up = vld1q_s8(upBuff1 + x);
+            int8x16_t sign_down = signOf_neon(rec + x, rec + x + stride - 1);
+
+            // Subtract instead of add, as sign_up is negated.
+            int8x16_t edge_type = vsubq_s8(sign_down, sign_up);
+
+            // For reuse in the next iteration.
+            vst1q_s8(upBuff1 + x - 1, sign_down);
+
+            edge_type = x265_sve_mask(x, endX, edge_type);
+            compute_eo_stats(edge_type, diff + x, tmp_count, tmp_stats);
+        }
+
+        upBuff1[endX - 1] = x265_signOf(rec[endX] - rec[endX - 1 + stride]);
+
+        rec += stride;
+        diff += MAX_CU_SIZE;
+    }
+
+    reduce_eo_stats(tmp_stats, tmp_count, stats, count);
+}
+
+void setupSaoPrimitives_sve(EncoderPrimitives &p)
+{
+    p.saoCuStatsE0 = saoCuStatsE0_sve;
+    p.saoCuStatsE1 = saoCuStatsE1_sve;
+    p.saoCuStatsE2 = saoCuStatsE2_sve;
+    p.saoCuStatsE3 = saoCuStatsE3_sve;
+}
+} // namespace X265_NS
diff --git a/source/common/aarch64/sao-prim.cpp b/source/common/aarch64/sao-prim.cpp
index 829ff297d..b3eda6bac 100644
--- a/source/common/aarch64/sao-prim.cpp
+++ b/source/common/aarch64/sao-prim.cpp
@@ -25,33 +25,6 @@
 #include "sao.h"
 #include <arm_neon.h>
 
-static inline int8x16_t signOf_neon(const pixel *a, const pixel *b)
-{
-#if HIGH_BIT_DEPTH
-    uint16x8_t s0_lo = vld1q_u16(a);
-    uint16x8_t s0_hi = vld1q_u16(a + 8);
-    uint16x8_t s1_lo = vld1q_u16(b);
-    uint16x8_t s1_hi = vld1q_u16(b + 8);
-
-    // signOf(a - b) = -(a > b) | (b > a)
-    int16x8_t cmp0_lo = vreinterpretq_s16_u16(vcgtq_u16(s0_lo, s1_lo));
-    int16x8_t cmp0_hi = vreinterpretq_s16_u16(vcgtq_u16(s0_hi, s1_hi));
-    int16x8_t cmp1_lo = vreinterpretq_s16_u16(vcgtq_u16(s1_lo, s0_lo));
-    int16x8_t cmp1_hi = vreinterpretq_s16_u16(vcgtq_u16(s1_hi, s0_hi));
-
-    int8x16_t cmp0 = vcombine_s8(vmovn_s16(cmp0_lo), vmovn_s16(cmp0_hi));
-    int8x16_t cmp1 = vcombine_s8(vmovn_s16(cmp1_lo), vmovn_s16(cmp1_hi));
-#else // HIGH_BIT_DEPTH
-    uint8x16_t s0 = vld1q_u8(a);
-    uint8x16_t s1 = vld1q_u8(b);
-
-    // signOf(a - b) = -(a > b) | (b > a)
-    int8x16_t cmp0 = vreinterpretq_s8_u8(vcgtq_u8(s0, s1));
-    int8x16_t cmp1 = vreinterpretq_s8_u8(vcgtq_u8(s1, s0));
-#endif // HIGH_BIT_DEPTH
-    return vorrq_s8(vnegq_s8(cmp0), cmp1);
-}
-
 // Predicate mask indices.
 static const int8_t quad_reg_byte_indices[16] = {
     0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15
diff --git a/source/common/aarch64/sao-prim.h b/source/common/aarch64/sao-prim.h
index 1437ccd75..e01dd28a5 100644
--- a/source/common/aarch64/sao-prim.h
+++ b/source/common/aarch64/sao-prim.h
@@ -25,9 +25,41 @@
 #define X265_COMMON_AARCH64_SAO_PRIM_H
 
 #include "primitives.h"
+#include <arm_neon.h>
+
+static inline int8x16_t signOf_neon(const pixel *a, const pixel *b)
+{
+#if HIGH_BIT_DEPTH
+    uint16x8_t s0_lo = vld1q_u16(a);
+    uint16x8_t s0_hi = vld1q_u16(a + 8);
+    uint16x8_t s1_lo = vld1q_u16(b);
+    uint16x8_t s1_hi = vld1q_u16(b + 8);
+
+    // signOf(a - b) = -(a > b) | (b > a)
+    int16x8_t cmp0_lo = vreinterpretq_s16_u16(vcgtq_u16(s0_lo, s1_lo));
+    int16x8_t cmp0_hi = vreinterpretq_s16_u16(vcgtq_u16(s0_hi, s1_hi));
+    int16x8_t cmp1_lo = vreinterpretq_s16_u16(vcgtq_u16(s1_lo, s0_lo));
+    int16x8_t cmp1_hi = vreinterpretq_s16_u16(vcgtq_u16(s1_hi, s0_hi));
+
+    int8x16_t cmp0 = vcombine_s8(vmovn_s16(cmp0_lo), vmovn_s16(cmp0_hi));
+    int8x16_t cmp1 = vcombine_s8(vmovn_s16(cmp1_lo), vmovn_s16(cmp1_hi));
+#else // HIGH_BIT_DEPTH
+    uint8x16_t s0 = vld1q_u8(a);
+    uint8x16_t s1 = vld1q_u8(b);
+
+    // signOf(a - b) = -(a > b) | (b > a)
+    int8x16_t cmp0 = vreinterpretq_s8_u8(vcgtq_u8(s0, s1));
+    int8x16_t cmp1 = vreinterpretq_s8_u8(vcgtq_u8(s1, s0));
+#endif // HIGH_BIT_DEPTH
+    return vorrq_s8(vnegq_s8(cmp0), cmp1);
+}
 
 namespace X265_NS {
 void setupSaoPrimitives_neon(EncoderPrimitives &p);
+
+#if defined(HAVE_SVE) && HAVE_SVE_BRIDGE
+void setupSaoPrimitives_sve(EncoderPrimitives &p);
+#endif
 }
 
 #endif // X265_COMMON_AARCH64_SAO_PRIM_H
-- 
2.42.1

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0006-AArch64-Add-SVE-saoCuStats-primitives.patch
Type: text/x-patch
Size: 19732 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20240520/5be0b40b/attachment-0001.bin>


More information about the x265-devel mailing list