2016-01-29 09:40, Jerin Jacob: > --- a/app/test/test_xmmt_ops.h > +++ b/app/test/test_xmmt_ops.h > +#if defined(RTE_ARCH_ARM) || defined(RTE_ARCH_ARM64) [...] > +#else [...] > --- a/lib/librte_lpm/Makefile > +++ b/lib/librte_lpm/Makefile > +ifneq ($(filter y,$(CONFIG_RTE_ARCH_ARM) $(CONFIG_RTE_ARCH_ARM64)),) > +SYMLINK-$(CONFIG_RTE_LIBRTE_LPM)-include += rte_lpm_neon.h > +else > SYMLINK-$(CONFIG_RTE_LIBRTE_LPM)-include += rte_lpm_sse.h > +endif [...] > --- a/lib/librte_lpm/rte_lpm.h > +++ b/lib/librte_lpm/rte_lpm.h > +#if defined(RTE_ARCH_ARM) || defined(RTE_ARCH_ARM64) > +#include "rte_lpm_neon.h" > +#else > #include "rte_lpm_sse.h" > +#endif
Instead of defaulting to x86 SSE, it would be better to replace "else" by "elif X86/SSE". I suggest using RTE_ARCH_X86 or RTE_CPUFLAG_SSEx. By the way, what is the minimum SSE version required?