On 8 October 2014 18:27, <charles.bay...@linaro.org> wrote:
> +#define __LD2_LANE_FUNC(intype, vectype, largetype, ptrtype, \
Just above the insertion point in arm-neon.h is the comment:
/* Start of temporary inline asm for vldn, vstn and friends. */
This patch removes the "temporary inline asm vldn" implementation, the
replacement implementation should be inserted in the section below,
look for the comment that starts:
"Start of optimal implementations...."
> + mode, ptrmode, funcsuffix, signedtype) \
> +__extension__ static __inline intype __attribute__ ((__always_inline__)) \
> +vld2_lane_##funcsuffix (const ptrtype * __ptr, intype __b, const int __c) \
> +{ \
> + __builtin_aarch64_simd_oi __o; \
> + largetype __temp; \
> + __temp.val[0] = \
There is something odd about the white space here, space before tab?
This is repeated in various places through the rest of the patch.
Otherwise this and the previous 1/2 associated patch look good, can
you respin with these tidy ups?
Thanks
/Marcus
