Andrew Pinski <pins...@gmail.com> writes: > > Using the builtin here seems wrong. Why not use the intrinsic > _mm256_movemask_epi8 ?
I followed the rest of the vectorized code paths. The original reason was that there was some incompatibility of the intrinsic header with the source build. I don't know if it's still true, but I guess it doesn't hurt. > Also it might make sense to remove the MMX version. See the previous patch. -Andi