[PR] tests/checkasm/Makefile: Don't build empty files (PR #24153)

PR #24153 opened by mkver
URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/24153
Patch URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/24153.patch

Empty translation units are actually invalid C that compilers
happen to accept (but warn about when using -pedantic).
In any case, they are unnecessary, so don't build them.



>From 1ef552e98ceb5e43a33517d6ef98e90eb1e7cf37 Mon Sep 17 00:00:00 2001
From: Andreas Rheinhardt <[email protected]>
Date: Fri, 14 Aug 2026 23:58:05 +0200
Subject: [PATCH] tests/checkasm/Makefile: Don't build empty files

Empty translation units are actually invalid C that compilers
happen to accept (but warn about when using -pedantic).
In any case, they are unnecessary, so don't build them.

Signed-off-by: Andreas Rheinhardt <[email protected]>
---
 tests/checkasm/Makefile | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/tests/checkasm/Makefile b/tests/checkasm/Makefile
index 169b40413b..24e2110aad 100644
--- a/tests/checkasm/Makefile
+++ b/tests/checkasm/Makefile
@@ -110,26 +110,26 @@ AVUTILOBJS-$(CONFIG_PIXELUTILS)         += pixelutils.o
 
 CHECKASMOBJS-$(CONFIG_AVUTIL)  += $(AVUTILOBJS) $(AVUTILOBJS-yes)
 
-EXT_CHECKASMOBJS-$(ARCH_AARCH64)            += ext/src/arm/checkasm_64.o
-EXT_CHECKASMOBJS-$(ARCH_ARM)                += ext/src/arm/checkasm_32.o
+EXT_CHECKASMOBJS-$(ARCH_AARCH64)            += ext/src/arm/checkasm_64.o \
+                                               ext/src/arm/cpu.o ext/src/perf/arm.o
+EXT_CHECKASMOBJS-$(ARCH_ARM)                += ext/src/arm/checkasm_32.o \
+                                               ext/src/arm/cpu.o ext/src/perf/arm.o
 EXT_CHECKASMOBJS-$(ARCH_LOONGARCH)          += ext/src/loongarch/checkasm.o
+EXT_CHECKASMOBJS-$(CONFIG_LINUX_PERF)       += ext/src/perf/linux.o
+EXT_CHECKASMOBJS-$(CONFIG_MACOS_KPERF)      += ext/src/perf/macos_kperf.o
 EXT_CHECKASMOBJS-$(HAVE_RV)                 += ext/src/riscv/callcheck.o
+EXT_CHECKASMOBJS-$(ARCH_RISCV)              += ext/src/riscv/cpu.o
 EXT_CHECKASMOBJS-$(HAVE_X86ASM)             += ext/src/x86/checkasm.o
+EXT_CHECKASMOBJS-$(ARCH_X86)                += ext/src/x86/cpu.o
 
-EXT_CHECKASMOBJS += ext/src/arm/cpu.o                  \
-                    ext/src/checkasm.o                 \
+EXT_CHECKASMOBJS += ext/src/checkasm.o                 \
                     ext/src/cpu.o                      \
                     ext/src/function.o                 \
                     ext/src/perf.o                     \
-                    ext/src/perf/arm.o                 \
-                    ext/src/perf/linux.o               \
-                    ext/src/perf/macos_kperf.o         \
-                    ext/src/riscv/cpu.o                \
                     ext/src/signal.o                   \
                     ext/src/stackguard.o               \
                     ext/src/stats.o                    \
                     ext/src/utils.o                    \
-                    ext/src/x86/cpu.o                  \
 
 EXT_CHECKASMOBJS += $(EXT_CHECKASMOBJS-yes)
 $(EXT_CHECKASMOBJS:%=tests/checkasm/%): CFLAGS      += -I$(SRC_PATH)/tests/checkasm/ext/src
-- 
2.52.0

_______________________________________________
ffmpeg-devel mailing list -- [email protected]
To unsubscribe send an email to [email protected]