From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v2] ACL: fix build for native-icc target on haswell fails Date: Fri, 20 Nov 2015 17:15:51 +0100 Message-ID: <4197524.MhqdmjAr8F@xps13> References: <1448033930-22524-1-git-send-email-konstantin.ananyev@intel.com> <564F436A.9010202@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org To: Konstantin Ananyev Return-path: Received: from mail-wm0-f47.google.com (mail-wm0-f47.google.com [74.125.82.47]) by dpdk.org (Postfix) with ESMTP id 94E628DB4 for ; Fri, 20 Nov 2015 17:17:05 +0100 (CET) Received: by wmww144 with SMTP id w144so27389671wmw.0 for ; Fri, 20 Nov 2015 08:17:05 -0800 (PST) In-Reply-To: <564F436A.9010202@intel.com> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 2015-11-20 15:59, Declan Doherty: > On 20/11/15 15:38, Konstantin Ananyev wrote: > > On HSW box with icc 16.0.0 build for x86_64-native-linuxapp-icc fails with: > > icc: command line warning #10120: overriding '-march=native' with '-msse4.1' > > ... > > dpdk.org/x86_64-native-linuxapp-icc/include/rte_memcpy.h(96): error: identifier "__m256i" is undefined > > > > The reason is that icc treats "-march=native ... -msse4.1" > > in a different way, then gcc and clang. > > For icc it means override all flags enabled with > > '-march=native' with '-msse4.1'. > > Even when '-march=native' is a superset for '-msse4.1'. > > To overcome the problem add a check is SSE4.1 compilation flag already enabled. > > If yes, then no need to add '-msse4.1' > > Similar change for avx2 compilation option. > > > > Fixes: 074f54ad03ee ("acl: fix build and runtime for default target") > > > > Reported-by: Declan Doherty > > Reported-by: Sergio Gonzalez Monroy > > Signed-off-by: Konstantin Ananyev > > Acked-by: Declan Doherty Applied, thanks