FM10K/dpdk-fm10k
FM10K/dpdk-fm10k
1
0
Fork 0
Code Issues Pull requests Releases Activity

mk: introduce helper to check compiler argument

Browse source 
Introduce rte_cc_has_argument() Makefile helper to
check a given argument is support by the compiler.

Example Usage:

include $(RTE_SDK)/mk/rte.helper.mk
MACHINE_CFLAGS += $(call rte_cc_has_argument, -mcpu=octeontx2)

This would allow adding -mcpu=octeontx2 in MACHINE_CFLAGS
if it is only supported by the compiler. The use case for such
scheme is to enable the mcpu optimization if the compiler
supports else it needs to compile the source code without
any errors.

This patch also moves inclusion of toolchain's rte.vars.mk
to before the machine's rte.vars.mk inclusion to make
correct CC available for the cross compile case.

Signed-off-by: Jerin Jacob <jerinj@marvell.com>
Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>
This commit is contained in:
Jerin Jacob 2019-04-14 01:49:43 +05:30 committed by Thomas Monjalon
parent 90cb27276a
commit 53a39551db
2 changed files with 21 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
mk/rte.helper.mk Normal file
View file

@ -0,0 +1,10 @@
# SPDX-License-Identifier: BSD-3-Clause
# Copyright(c) 2018 Marvell International Ltd
# rte_cc_has_argument
# Usage: MACHINE_CFLAGS += $(call rte_cc_has_argument, -mno-avx512f)
# Return the argument if the argument is supported by the compiler.
#
define rte_cc_has_argument
$(shell $(CC) -E $(1) -xc /dev/null 1>/dev/null 2>/dev/null && echo $(1))
endef

22
mk/target/generic/rte.vars.mk
View file

@ -7,6 +7,17 @@
# executive environment.
#
#
# toolchain:
#
# - define CC, LD, AR, AS, ...
# - define TOOLCHAIN_CFLAGS variable (overridden by cmdline value)
# - define TOOLCHAIN_LDFLAGS variable (overridden by cmdline value)
# - define TOOLCHAIN_ASFLAGS variable (overridden by cmdline value)
# - may override any previously defined variable
#
include $(RTE_SDK)/mk/toolchain/$(RTE_TOOLCHAIN)/rte.vars.mk
#
# machine:
#
@ -45,17 +56,6 @@ endif
#
include $(RTE_SDK)/mk/arch/$(RTE_ARCH)/rte.vars.mk
#
# toolchain:
#
# - define CC, LD, AR, AS, ...
# - define TOOLCHAIN_CFLAGS variable (overridden by cmdline value)
# - define TOOLCHAIN_LDFLAGS variable (overridden by cmdline value)
# - define TOOLCHAIN_ASFLAGS variable (overridden by cmdline value)
# - may override any previously defined variable
#
include $(RTE_SDK)/mk/toolchain/$(RTE_TOOLCHAIN)/rte.vars.mk
#
# exec-env:
#