On 12/19/2018 8:28 PM, Thomas Monjalon wrote: > 19/12/2018 21:20, Ferruh Yigit: >> On 12/19/2018 7:58 PM, Thomas Monjalon wrote: >>> 19/12/2018 20:29, 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. >>> [...] >>>> # disable AVX512F support of gcc as a workaround for Bug 97 >>>> ifeq ($(CONFIG_RTE_TOOLCHAIN_GCC),y) >>>> +ifneq ($(filter 2.30%,$(LD_VERSION)),) >>>> MACHINE_CFLAGS += -mno-avx512f >>> >>> I think we should print a warning here. >>> There is a function $(warning) or $(info) for such case. >> >> I can add but it prints warning per component, so it prints a lot. > > I think we can choose to print only when compiling EAL > by checking $(findstring eal,$(lastword $(MAKEFILE_LIST)))
Thanks for hint, I will use it. > > One more comment: it would be good to have this fix with meson too. Agreed, only I am not quite sure how to fix it in meson.
