[FFmpeg-devel] [PATCH] checkasm/lls: adjust buffer sizes and alignments

Rémi Denis-Courmont remi at remlab.net
Sat Jun 8 09:35:42 EEST 2024


var must be padded.
param has `order + 1`, not `order` elements and is *not* over-aligned.
---
 tests/checkasm/lls.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/tests/checkasm/lls.c b/tests/checkasm/lls.c
index a2e8f11f99..b1db21cd56 100644
--- a/tests/checkasm/lls.c
+++ b/tests/checkasm/lls.c
@@ -82,8 +82,8 @@ void checkasm_check_lls(void)
     static const unsigned char counts[] = { 8, 12, MAX_VARS, };
 
     for (size_t i = 0; i < FF_ARRAY_ELEMS(counts); i++) {
-        LOCAL_ALIGNED_32(double, var, [MAX_VARS]);
-        LOCAL_ALIGNED_32(double, param, [MAX_VARS]);
+        LOCAL_ALIGNED_32(double, var, [MAX_VARS_ALIGN]);
+        LOCAL_ALIGNED_32(double, param, [FFALIGN(MAX_VARS+2,4)]);
         LLSModel lls;
 
         avpriv_init_lls(&lls, counts[i]);
@@ -95,7 +95,7 @@ void checkasm_check_lls(void)
         for (size_t j = 0; j <= i; j++)
             if (check_func(lls.evaluate_lls, "evaluate_lls_%d_%d", counts[i],
                            counts[j]))
-                test_evaluate(&lls, param, counts[j]);
+                test_evaluate(&lls, param + 1, counts[j]);
     }
     report("lls");
 }
-- 
2.45.1



More information about the ffmpeg-devel mailing list