Re: [FFmpeg-devel] [PATCH] configure: set IceLake-AVX512 as the minimum baseline

Ronald S. Bultje Wrote: > I'm OK with the current approach also. So I will not object to the current > patch anymore. > Hi Ronald, I am sorry I'm not familiar of the review flow. Does this proposal still need to be reviewed and approved by other maintainers? Or if it's acceptable, I added the

Re: [FFmpeg-devel] [PATCH] configure: set IceLake-AVX512 as the minimum baseline

Hi Jianhua, On Wed, Aug 18, 2021 at 7:31 AM qs xw wrote: > So what should I do? Feel free to let me know. > My personal slight preference would be to continue allowing Skylake-style asm, i.e. add a new runtime flag for ICL-style avx512. Then, people on skylake can at their choice disable avx512

Re: [FFmpeg-devel] [PATCH] configure: set IceLake-AVX512 as the minimum baseline

Ronald Wrote: > On Tue, Aug 17, 2021 at 10:13 PM Wu, Jianhua intel.com> wrote: > > The reason why we set the icelake-avx512 as the minimum baseline is that > > we don't want FFmpeg run on a processor downclocking heavily, hence, keep > > the

Re: [FFmpeg-devel] [PATCH] configure: set IceLake-AVX512 as the minimum baseline

Hello, On 2021-08-18 12:40, Ronald S. Bultje wrote: I don't think you can make that decision single-handedly. doesn't it depend on the situation, and shouldn't individual users and/or developers be able to make that decision themselves? I agree, not all AVX512 instructions slow down those

Re: [FFmpeg-devel] [PATCH] configure: set IceLake-AVX512 as the minimum baseline

Hi, On Tue, Aug 17, 2021 at 10:13 PM Wu, Jianhua wrote: > The reason why we set the icelake-avx512 as the minimum baseline is that > we don't want FFmpeg run on a processor downclocking heavily, hence, keep > the current flag and add a new AVX512ICL make no sense. > I don't think you can make t

Re: [FFmpeg-devel] [PATCH] configure: set IceLake-AVX512 as the minimum baseline

James Almer W rote: > On 8/17/2021 4:24 PM, James Almer wrote: > > On 8/17/2021 12:25 PM, Ronald S. Bultje wrote: > >> Hi, > >> > >> On Tue, Aug 17, 2021 at 2:33 AM Hendrik Leppkes > >> wrote: > >> > >>> On Tue, Aug 17, 2021 at 8:30 AM Wu Jianhua > wrote: > Based on IceLake-AVX512 and new

Re: [FFmpeg-devel] [PATCH] configure: set IceLake-AVX512 as the minimum baseline

Hi, On Tue, Aug 17, 2021 at 3:27 PM James Almer wrote: > On 8/17/2021 4:24 PM, James Almer wrote: > > On 8/17/2021 12:25 PM, Ronald S. Bultje wrote: > >> On Tue, Aug 17, 2021 at 2:33 AM Hendrik Leppkes > >> wrote: > >>> On Tue, Aug 17, 2021 at 8:30 AM Wu Jianhua > wrote: > Based on IceLak

Re: [FFmpeg-devel] [PATCH] configure: set IceLake-AVX512 as the minimum baseline

On 8/17/2021 4:24 PM, James Almer wrote: On 8/17/2021 12:25 PM, Ronald S. Bultje wrote: Hi, On Tue, Aug 17, 2021 at 2:33 AM Hendrik Leppkes wrote: On Tue, Aug 17, 2021 at 8:30 AM Wu Jianhua wrote: Based on IceLake-AVX512 and newer architecture, a broad range of the subsets of AVX512 coul

Re: [FFmpeg-devel] [PATCH] configure: set IceLake-AVX512 as the minimum baseline

On 8/17/2021 12:25 PM, Ronald S. Bultje wrote: Hi, On Tue, Aug 17, 2021 at 2:33 AM Hendrik Leppkes wrote: On Tue, Aug 17, 2021 at 8:30 AM Wu Jianhua wrote: Based on IceLake-AVX512 and newer architecture, a broad range of the subsets of AVX512 could be supported. [..] -enabled a

Re: [FFmpeg-devel] [PATCH] configure: set IceLake-AVX512 as the minimum baseline

Hi, On Tue, Aug 17, 2021 at 2:33 AM Hendrik Leppkes wrote: > On Tue, Aug 17, 2021 at 8:30 AM Wu Jianhua wrote: > > Based on IceLake-AVX512 and newer architecture, a broad > > range of the subsets of AVX512 could be supported. > [..] > > -enabled avx512 && check_x86asm avx512_external "

Re: [FFmpeg-devel] [PATCH] configure: set IceLake-AVX512 as the minimum baseline

On Tue, Aug 17, 2021 at 8:30 AM Wu Jianhua wrote: > > Based on IceLake-AVX512 and newer architecture, a broad > range of the subsets of AVX512 could be supported. > > Signed-off-by: Wu Jianhua > --- > configure | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/configure

[FFmpeg-devel] [PATCH] configure: set IceLake-AVX512 as the minimum baseline

Based on IceLake-AVX512 and newer architecture, a broad range of the subsets of AVX512 could be supported. Signed-off-by: Wu Jianhua --- configure | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/configure b/configure index 94b30afe74..04caa25736 100755 --- a/configure +++