14/01/2019 16:49, Ferruh Yigit:
> AVX512 was disabled for GCC because of Bugzilla issue 97 [1],
> the GCC defect submitted for the issue [2] highlighted that this is
> a known binutils version 2.30 issue.
> 
> Narrowed the scope of no-avx512 to the this specific binutils version.
> 
> [1]
> https://bugs.dpdk.org/show_bug.cgi?id=97
> 
> [2]
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88096
> 
> Fixes: 8d07c82b239f ("mk: disable gcc AVX512F support")
> Cc: [email protected]
> 
> Signed-off-by: Ferruh Yigit <[email protected]>
> ---
> Cc: Tom Barbette <[email protected]>
> Cc: Thomas Monjalon <[email protected]>
> Cc: Yongseok Koh <[email protected]>
> Cc: Konstantin Ananyev <[email protected]>
> Cc: Bruce Richardson <[email protected]>
> 
> v2:
> * Added warning message (print only once for eal)
> * Moved decision details to compat file, kept execution in cpuflags
> 
> v3:
> * replace 'ld' with '$(LD)' for cross build
> * added meson support
> 
> v4:
> * Document this limitation in release notes
> Cc: Vipin Varghese <[email protected]>
> 
> v5:
> * Grammar fix
> * remove $(lastword.. ) from makefile
> * add 'OR' between possible workarounds

Acked-by: Thomas Monjalon <[email protected]>

Applied, thanks



Reply via email to