Re: [FFmpeg-devel] [PATCH 03/11] avutil: detect when AVX-512 is available

2017-11-20 Thread James Darnley
On 2017-11-10 03:11, James Almer wrote: > On 11/9/2017 8:58 AM, James Darnley wrote: >> @@ -154,6 +155,13 @@ int ff_get_cpu_flags_x86(void) >> if (ebx & 0x0100) >> rval |= AV_CPU_FLAG_BMI2; >> } >> +#if HAVE_AVX512 /* F, CD, BW, DQ, VL */ > > Nit: Maybe

Re: [FFmpeg-devel] [PATCH 03/11] avutil: detect when AVX-512 is available

2017-11-09 Thread James Almer
On 11/9/2017 8:58 AM, James Darnley wrote: > --- > I've changed this patch slightly because I discovered that it would cause an > illegal instruction exception on much older processors (probably all without > AVX). I was running xgetbv() almost uncontitionally. Now it is a little more > like

[FFmpeg-devel] [PATCH 03/11] avutil: detect when AVX-512 is available

2017-11-09 Thread James Darnley
--- I've changed this patch slightly because I discovered that it would cause an illegal instruction exception on much older processors (probably all without AVX). I was running xgetbv() almost uncontitionally. Now it is a little more like what is the in x264 patch. libavutil/x86/cpu.c | 12