commit 7fc3fa6899600f2356c471632dc464992f965d78
author James Zern <jzern@google.com> Tue May 28 15:58:26 2024 -0700
committer James Zern <jzern@google.com> Wed May 29 20:13:20 2024 +0000
tree 8b0b45e73c26d97f91e5c31cb9659bc45f7e2df7
parent 4e1a5d4d08a216d71ecee10c25736abb2ffc2cdc

av1_block_error_lp_sve: fix block_size param type

int -> intptr_t. This fixes a Control Flow Integrity (CFI) sanitizer
failure.

This also fixes a -Wmissing-prototypes warning. The _neon version was
fixed in:
b44333201b *_neon.c: add missing rtcd includes & CONFIG check

Bug: aomedia:3416
Change-Id: Iecf4a0f450435d6afa481695e000bcc0c8f4079c

av1/encoder/arm/av1_error_sve.c

diff --git a/av1/encoder/arm/av1_error_sve.c b/av1/encoder/arm/av1_error_sve.c
index 52803a9..5a1ad2f 100644
--- a/av1/encoder/arm/av1_error_sve.c
+++ b/av1/encoder/arm/av1_error_sve.c
@@ -12,6 +12,7 @@
 #include <assert.h>
 
 #include "config/aom_config.h"
+#include "config/av1_rtcd.h"
 
 #include "aom_dsp/aom_dsp_common.h"
 #include "aom_dsp/arm/aom_neon_sve_bridge.h"
@@ -49,7 +50,7 @@
 }
 
 int64_t av1_block_error_lp_sve(const int16_t *coeff, const int16_t *dqcoeff,
-                               int block_size) {
+                               intptr_t block_size) {
   if (block_size % 32 == 0) {
     int64x2_t error[4] = { vdupq_n_s64(0), vdupq_n_s64(0), vdupq_n_s64(0),
                            vdupq_n_s64(0) };