Re: [ovs-dev] [PATCH v4 5/5] acinclude: Add seperate checks for AVX512 ISA.

2022-05-17 Thread Ferriter, Cian
> -Original Message- > From: Pai G, Sunil > Sent: Thursday 5 May 2022 11:26 > To: Ferriter, Cian ; ovs-dev@openvswitch.org > Cc: Van Haaren, Harry > Subject: RE: [PATCH v4 5/5] acinclude: Add seperate checks for AVX512 ISA. > > Hi Cian, > > Few comments inline. > > > Hi Sunil,

Re: [ovs-dev] [PATCH v4 5/5] acinclude: Add seperate checks for AVX512 ISA.

2022-05-05 Thread Pai G, Sunil
Hi Cian, Few comments inline. > diff --git a/acinclude.m4 b/acinclude.m4 index 61e88105f..7b2889a40 100644 > --- a/acinclude.m4 > +++ b/acinclude.m4 > @@ -73,16 +73,13 @@ AC_DEFUN([OVS_CHECK_DPIF_AVX512_DEFAULT], [ > > dnl OVS_CHECK_AVX512 > dnl > -dnl Checks if compiler and binutils

[ovs-dev] [PATCH v4 5/5] acinclude: Add seperate checks for AVX512 ISA.

2022-04-29 Thread Cian Ferriter
Checking for each of the required AVX512 ISA separately will allow the compiler to generate some AVX512 code where there is some support in the compiler rather than only generating all AVX512 code when all of it is supported or no AVX512 code at all. For example, in GCC 4.9 where there is just