[dpdk-dev] [PATCH v4 12/14] acl: deduplicate AVX512 code paths

[Thomas Monjalon]

16/10/2020 17:56, Ferruh Yigit:
> On 10/6/2020 4:03 PM, Konstantin Ananyev wrote:
> > @@ -120,7 +161,7 @@ _m256_mask_gather_epi8x4(__m256i pdata, __mmask8 mask)
> >   
> >   	static const uint32_t zero;
> 
> icc complains about this, although it is static [1].
> Would it be acceptable to initialize variable explicitly to '0'?
> 
> [1]
> In file included from ../lib/librte_acl/acl_run_avx512.c(110):
> ../lib/librte_acl/acl_run_avx512x8.h(162): warning #300: const variable "zero" 
> requires an initializer
>          static const uint32_t zero;

icc should be fixed in my opinion

It's already hard to have code warning-free for gcc and clang,
I don't think we should bother with warnings from exotic compilers.
It is not a blocking issue, right?