[FFmpeg-devel] [PATCH 9/9] checkasm/idctdsp: Use declare_func_emms only when needed

Andreas Rheinhardt andreas.rheinhardt at outlook.com
Sat Oct 8 04:24:12 EEST 2022


There is no MMX code for (add|put|put_signed)_pixels_clamped
since commit bfb28b5ce89f3e950214b67ea95b45e3355c2caf, so use
declare_func instead of declare_func_emms() to also test that
we are not in MMX mode after return.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>
---
 tests/checkasm/idctdsp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/checkasm/idctdsp.c b/tests/checkasm/idctdsp.c
index 02724536a7..c06e607d99 100644
--- a/tests/checkasm/idctdsp.c
+++ b/tests/checkasm/idctdsp.c
@@ -79,7 +79,7 @@ static void check_add_put_clamped(void)
     for (size_t t = 0; t < FF_ARRAY_ELEMS(tests); ++t) {
         void (*func)(const int16_t *, uint8_t * ptrdiff_t) = *(void **)((intptr_t) &h + tests[t].offset);
         if (check_func(func, "idctdsp.%s", tests[t].name)) {
-            declare_func_emms(AV_CPU_FLAG_MMX, void, const int16_t *, uint8_t *, ptrdiff_t);
+            declare_func(void, const int16_t *, uint8_t *, ptrdiff_t);
             RANDOMIZE_BUFFER16(src, 64);
             RANDOMIZE_BUFFER8(dst, 10 * 24);
             call_ref(src0, dst0 + 24 + 8, 24);
-- 
2.34.1



More information about the ffmpeg-devel mailing list