On Tue, 1 Dec 2015 20:13:49 +0530 Jerin Jacob <jerin.jacob at caviumnetworks.com> wrote:
> > enum rte_acl_classify_alg alg = RTE_ACL_CLASSIFY_DEFAULT; > > > > -#ifdef RTE_ARCH_ARM64 > > +#if defined(RTE_ARCH_ARM) || defined(RTE_ARCH_ARM64) > > alg = RTE_ACL_CLASSIFY_NEON; > > I believe SIMD is optional in armv7. If true, select alg as > RTE_ACL_CLASSIFY_NEON only when cpufeature NEON enabled. Yes. Or, probably, we can be happy with #if defined(__ARM_NEON_FP) ... #endif as it is currently done in rte_memcpy_32.h. Regards Jan