Jan 7, 2021, 00:13 by [email protected]: > Also do it for FFT_FLOAT only, as this is the only combination for which > it can be set. > > Signed-off-by: Andreas Rheinhardt <[email protected]> > --- > 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 [email protected] https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email [email protected] with subject "unsubscribe".
