From f27769f796a0639368117ce22fb124b6030dbf73 Mon Sep 17 00:00:00 2001 From: Bruce Richardson Date: Tue, 20 Jun 2017 16:22:56 +0100 Subject: [PATCH] mk: require SSE4.2 support on all x86 platforms Increase the default baseline from "core2" architecture to "corei7". This means that all builds will have SSE4.2 support included, and we can remove special case manipulation of CFLAGS for the same. Naturally, this does mean that some machines that previously could run DPDK now can't do so, but hardware with SSE4.2 has been around for almost a decade now, so this should not be a major problem. Signed-off-by: Bruce Richardson Acked-by: Konstantin Ananyev --- mk/machine/default/rte.vars.mk | 2 +- mk/machine/native/rte.vars.mk | 3 --- mk/toolchain/icc/rte.toolchain-compat.mk | 2 +- 3 files changed, 2 insertions(+), 5 deletions(-) diff --git a/mk/machine/default/rte.vars.mk b/mk/machine/default/rte.vars.mk index a6fb84244b..71a25bcae3 100644 --- a/mk/machine/default/rte.vars.mk +++ b/mk/machine/default/rte.vars.mk @@ -55,4 +55,4 @@ # CPU_LDFLAGS = # CPU_ASFLAGS = -MACHINE_CFLAGS += -march=core2 +MACHINE_CFLAGS += -march=corei7 diff --git a/mk/machine/native/rte.vars.mk b/mk/machine/native/rte.vars.mk index 7f55b54a89..f7d98d0dc9 100644 --- a/mk/machine/native/rte.vars.mk +++ b/mk/machine/native/rte.vars.mk @@ -63,8 +63,5 @@ MACHINE_CFLAGS = -march=native # so, set the compilation target to be a corei7, minimum target with SSE4.2. SSE42_SUPPORT=$(shell $(CC) -march=native -dM -E - /dev/null) - ifneq ($(CPU_SSE42_SUPPORT),) MACHINE_CFLAGS = -march=corei7 - endif endif diff --git a/mk/toolchain/icc/rte.toolchain-compat.mk b/mk/toolchain/icc/rte.toolchain-compat.mk index 88f1ac92f4..3c25d184c5 100644 --- a/mk/toolchain/icc/rte.toolchain-compat.mk +++ b/mk/toolchain/icc/rte.toolchain-compat.mk @@ -41,7 +41,7 @@ ICC_MAJOR_VERSION = $(shell icc -dumpversion | cut -f1 -d.) ifeq ($(shell test $(ICC_MAJOR_VERSION) -lt 12 && echo 1), 1) - MACHINE_CFLAGS = -xSSE3 + MACHINE_CFLAGS = -xSSE4.2 $(warning You are not using ICC 12.x or higher. This is neither supported, nor tested.) else -- 2.20.1