[FFmpeg-cvslog] checkasm/blockdsp: use smallest allowed aligned buffers for fill_block_tab tests

[James Almer]

ffmpeg | branch: master | James Almer <jamrial at gmail.com> | Tue May  7 11:58:55 2024 -0300| [b6054e57cb5fa7b6bad834dcfbf0f3dd934ba276] | committer: James Almer

checkasm/blockdsp: use smallest allowed aligned buffers for fill_block_tab tests

The requirement is either 8 or 16 bytes alignment, not 32.
This should help finding bugs in asm implementations.

Signed-off-by: James Almer <jamrial at gmail.com>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=b6054e57cb5fa7b6bad834dcfbf0f3dd934ba276
---

 tests/checkasm/blockdsp.c | 21 +++++++--------------
 1 file changed, 7 insertions(+), 14 deletions(-)

diff --git a/tests/checkasm/blockdsp.c b/tests/checkasm/blockdsp.c
index 1f02d02076..5f4d46b8fa 100644
--- a/tests/checkasm/blockdsp.c
+++ b/tests/checkasm/blockdsp.c
@@ -29,11 +29,6 @@
 #include "libavutil/intreadwrite.h"
 #include "libavutil/mem_internal.h"
 
-typedef struct {
-    const char *name;
-    int size;
-} test;
-
 #define randomize_buffers(size)             \
     do {                                    \
         int i;                              \
@@ -58,18 +53,16 @@ do {                                                                \
 } while (0)
 
 static void check_fill(BlockDSPContext *h){
-    const test tests[] = {
-        {"fill_block_tab[0]", 16},
-        {"fill_block_tab[1]", 8},
-    };
-    LOCAL_ALIGNED_32(uint8_t, buf0, [16 * 16]);
-    LOCAL_ALIGNED_32(uint8_t, buf1, [16 * 16]);
+    LOCAL_ALIGNED_16(uint8_t, buf0_16, [16 * 16]);
+    LOCAL_ALIGNED_16(uint8_t, buf1_16, [16 * 16]);
 
-    for (size_t t = 0; t < FF_ARRAY_ELEMS(tests); ++t) {
-        int n = tests[t].size;
+    for (int t = 0; t < 2; ++t) {
+        uint8_t *buf0 = buf0_16 + t * /* force 8 byte alignment */ 8;
+        uint8_t *buf1 = buf1_16 + t * /* force 8 byte alignment */ 8;
+        int n = 16 - 8 * t;
         declare_func(void, uint8_t *block, uint8_t value,
                      ptrdiff_t line_size, int h);
-        if (check_func(h->fill_block_tab[t], "blockdsp.%s", tests[t].name)) {
+        if (check_func(h->fill_block_tab[t], "blockdsp.fill_block_tab[%d]", t)) {
             uint8_t value = rnd();
             memset(buf0, 0, sizeof(*buf0) * n * n);
             memset(buf1, 0, sizeof(*buf1) * n * n);