2016-03-30 14:15, Dumitrescu, Cristian: > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > 2016-03-30 13:57, Dumitrescu, Cristian: > > > I think the correct fix is: > > > #if defined(__x86_64__) && (defined(RTE_MACHINE_CPUFLAG_SSE4_2) > > || defined(RTE_MACHINE_CPUFLAG_CRC32)) > > > > > > We'll test it and send a patch asap. > > > > I had prepared this patch. Please be inspired: > > > > examples/ip_pipeline: fix SSE4.2 optimization branch > > > > The branch was disabled because of a typo in the SSE4.2 flag. > > Change also the x86_64 flag to use a DPDK one. > > > > Fixes: 28377375c6c0 ("examples/ip_pipeline: fix build for x86_64 without > > SSE4.2") > > > > -#if defined(__x86_64__) && defined(RTE_CPUFLAG_SSE4_2) > > +#if defined(RTE_ARCH_X86_64) && > > defined(RTE_MACHINE_CPUFLAG_SSE4_2) > > Acked-by: Cristian Dumitrescu <cristian.dumitrescu at intel.com>
I thought you wanted to send a patch with another CPUFLAG (CRC32)?