> -----Original Message----- > From: Thomas Monjalon <tho...@monjalon.net> > Sent: Friday, May 21, 2021 4:11 PM > To: lian...@liangbit.com; Ananyev, Konstantin <konstantin.anan...@intel.com> > Cc: dev@dpdk.org; sta...@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: sta...@dpdk.org > > > > Reported-by: Liang Ma <lian...@liangbit.com> > > Signed-off-by: Konstantin Ananyev <konstantin.anan...@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.