[FFmpeg-devel] [PATCH v2 1/5] libavfilter/x86/vf_gblur: add ff_postscale_slice_avx512()

[Wu, Jianhua]

Kieran Wrote:
> Hi Jianhua,
> 
> Can you make the avx512 code only be run for ice lake processors?
> 

Hi Kieran,

As I see the macro, EXTERNAL_AVX512, located in libavutil/cpu.h, only an
EXTERNAL_AVX512  is used to check the state of AVX512, which did not
district the Icelake-AVX512 or Skylake-AVX512. If I have the approval from
maintainers, I'm supposed to be able to change the configure file and CPU.h
to add a feature that the configuration only enables AVX512 when the
architecture of a running CPU is satisfied to the IceLake. Or we could keep
the EXTERNAL_AVX512 and add a new macro, EXTERNAL_AVX512ICL. How
about your opinion?

By the way, these patches only used the minimum subsets of AVX512 and
have higher compatibility. I think it's okay to use it in any arch that supports
AVX512.

Feel free to let me know if you have different perspectives or if I neglected something.

Best regards.
Jianhua