Message ID | 20210106231308.2952217-4-andreas.rheinhardt@gmail.com |
---|---|
State | Accepted |
Commit | a454a0c14fa2c2bf712f282a7fcc574bdc90a327 |
Headers | show |
Series | [FFmpeg-devel,1/5] avcodec/tableprint: Don't include mem_internal.h | expand |
Context | Check | Description |
---|---|---|
andriy/x86_make | success | Make finished |
andriy/x86_make_fate | success | Make fate finished |
andriy/PPC64_make | success | Make finished |
andriy/PPC64_make_fate | success | Make fate finished |
Jan 7, 2021, 00:13 by andreas.rheinhardt@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.rheinhardt@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.
Lynne: > Jan 7, 2021, 00:13 by andreas.rheinhardt@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.rheinhardt@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. I don't see a reason to interfere in the compiler's inlining decision here. It is a static function only called once, so it will be inlined anyway. - Andreas
Jan 7, 2021, 17:38 by andreas.rheinhardt@gmail.com: > Lynne: > >> Jan 7, 2021, 00:13 by andreas.rheinhardt@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.rheinhardt@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. >> > > I don't see a reason to interfere in the compiler's inlining decision > here. It is a static function only called once, so it will be inlined > anyway. > Fair enough.
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 {\
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.rheinhardt@gmail.com> --- libavcodec/fft_template.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)