-----Original Message----- > Date: Wed, 22 Aug 2018 10:01:07 -0500 > From: Honnappa Nagarahalli <[email protected]> > To: [email protected] > CC: [email protected], [email protected], [email protected], > [email protected], [email protected] > Subject: [dpdk-dev] [PATCH] build: enable Arm NEON flags when __aarch64__ > is defined > X-Mailer: git-send-email 2.7.4 > > External Email > > GCC version 4.8.5 does not pre-define __ARM_NEON. NEON is not > optional for ArmV8. Hence NEON related code can be enabled > when __aarch64__ is defined. > > Bugzilla ID: 82 > > Signed-off-by: Honnappa Nagarahalli <[email protected]> > Reviewed-by: Phil Yang <[email protected]> > Reviewed-by: Gavin Hu <[email protected]> > Reported-by: Raslan Darawsheh <[email protected]> > Reported-by: Thomas F Herbert <[email protected]>
Fixes: Cc: stable blank line Reported-by: Suggested-by: Signed-off-by: In general, Please follow the above order. With above change: Acked-by: Jerin Jacob <[email protected]> > --- > config/arm/meson.build | 3 ++- > mk/rte.cpuflags.mk | 2 +- > 2 files changed, 3 insertions(+), 2 deletions(-) > > diff --git a/config/arm/meson.build b/config/arm/meson.build > index 40dbc87f7..94cca490e 100644 > --- a/config/arm/meson.build > +++ b/config/arm/meson.build > @@ -157,7 +157,8 @@ else > endif > message(machine_args) > > -if cc.get_define('__ARM_NEON', args: machine_args) != '' > +if (cc.get_define('__ARM_NEON', args: machine_args) != '' or > + cc.get_define('__aarch64__', args: machine_args) != '') > dpdk_conf.set('RTE_MACHINE_CPUFLAG_NEON', 1) > compile_time_cpuflags += ['RTE_CPUFLAG_NEON'] > endif > diff --git a/mk/rte.cpuflags.mk b/mk/rte.cpuflags.mk > index 60713137d..43ed84155 100644 > --- a/mk/rte.cpuflags.mk > +++ b/mk/rte.cpuflags.mk > @@ -89,7 +89,7 @@ CPUFLAGS += VSX > endif > > # ARM flags > -ifneq ($(filter $(AUTO_CPUFLAGS),__ARM_NEON),) > +ifneq ($(filter __ARM_NEON __aarch64__,$(AUTO_CPUFLAGS)),) > CPUFLAGS += NEON > endif > > -- > 2.17.1 >
