On Tue, Jul 18, 2017 at 7:57 AM, Michael Collison <michael.colli...@arm.com> wrote: > This is the second version of a patch for Aarc64 to add a vectorized mersenne > twister to libstdc++. The first version used intrinsics and included > "arm_neon.h". After feedback from the community this version uses only GCC > vector extensions and Aarch64 simd data types.
Looks OK. Just stylistically, why do you have +#ifdef __ARM_NEON +#ifdef __aarch64__ (in more than one place) instead of one preprocessor line?
