[FFmpeg-devel] [PATCH 1/2] lavc/aarch64: remove bogus HAVE_VFP guard

Rémi Denis-Courmont remi at remlab.net
Fri Jul 14 19:34:47 EEST 2023


The IMDCT offset is only relevant for NEON optimisations. There are no
VFP optimisations here that would justify the HAVE_VFP flag. In
practice, this makes no difference because HAVE_NEON is practically
always true for standard Armv8 platforms.
---
 libavcodec/aarch64/synth_filter_init.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavcodec/aarch64/synth_filter_init.c b/libavcodec/aarch64/synth_filter_init.c
index 801b46e217..6b6da35b54 100644
--- a/libavcodec/aarch64/synth_filter_init.c
+++ b/libavcodec/aarch64/synth_filter_init.c
@@ -28,7 +28,7 @@
 
 #include "asm-offsets.h"
 
-#if HAVE_NEON || HAVE_VFP
+#if HAVE_NEON
 AV_CHECK_OFFSET(FFTContext, imdct_half, IMDCT_HALF);
 #endif
 
-- 
2.40.1



More information about the ffmpeg-devel mailing list