Jan 7, 2021, 00:13 by andreas.rheinha...@gmail.com: > Also do it for FFT_FLOAT only, as this is the only combination for which > it can be set. > > Signed-off-by: Andreas Rheinhardt <andreas.rheinha...@gmail.com> > --- > libavcodec/fft_template.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/fft_template.c b/libavcodec/fft_template.c > index 9d125de073..ddde63714e 100644 > --- a/libavcodec/fft_template.c > +++ b/libavcodec/fft_template.c > @@ -251,7 +251,7 @@ av_cold int ff_fft_init(FFTContext *s, int nbits, int > inverse) > #endif /* FFT_FIXED_32 */ > > > - if (s->fft_permutation == FF_FFT_PERM_AVX) { > + if (ARCH_X86 && FFT_FLOAT && s->fft_permutation == FF_FFT_PERM_AVX) { > fft_perm_avx(s); > } else { > #define PROCESS_FFT_PERM_SWAP_LSBS(num) do {\ >
LGTM. Maybe mark fft_perm_avx as inline too if you can be bothered to amend the patch. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".