[dpdk-dev] [PATCH] acl: fix build with GCC 6.3

[Ananyev, Konstantin]

> -----Original Message-----
> From: Thomas Monjalon <thomas at monjalon.net>
> Sent: Friday, May 21, 2021 4:11 PM
> To: liangma at liangbit.com; Ananyev, Konstantin <konstantin.ananyev at intel.com>
> Cc: dev at dpdk.org; stable at dpdk.org
> Subject: Re: [dpdk-dev] [PATCH] acl: fix build with GCC 6.3
> 
> 21/05/2021 16:42, Konstantin Ananyev:
> > --buildtype=debug with gcc 6.3 produces the following error:
> >
> > ../lib/librte_acl/acl_run_avx512_common.h: In function
> > ‘resolve_match_idx_avx512x16’:
> > ../lib/librte_acl/acl_run_avx512x16.h:33:18: error:
> > 	the last argument must be an 8-bit immediate
> >                                ^
> > ../lib/librte_acl/acl_run_avx512_common.h:373:9: note:
> > 	in expansion of macro ‘_M_I_’
> >       return _M_I_(slli_epi32)(mi, match_log);
> >              ^~~~~
> >
> > Seems like gcc-6.3 complains about the following construct:
> >
> > static const uint32_t match_log = 5;
> >     ...
> > _mm512_slli_epi32(mi, match_log);
> >
> > It can't substitute constant variable 'match_log' with its actual value.
> > The fix replaces constant variable with its immediate value.
> >
> > Bugzilla ID: 717
> > Fixes: b64c2295f7fc ("acl: add 256-bit AVX512 classify method")
> > Fixes: 45da22e42ec3 ("acl: add 512-bit AVX512 classify method")
> > Cc: stable at dpdk.org
> >
> > Reported-by: Liang Ma <liangma at liangbit.com>
> > Signed-off-by: Konstantin Ananyev <konstantin.ananyev at intel.com>
> 
> How much critical is it?
> It looks safer to wait for 21.08 cycle?

I think we can wait till 21.08 cycle.