[PATCH] Warn on use of vector initializers in ARM BE mode

Tim Northover t.p.northover at gmail.com
Thu Jun 19 04:38:46 PDT 2014


Hi James,

On 19 June 2014 11:13, James Molloy <james.molloy at arm.com> wrote:
> Thanks for the comments. New patch attached, is it any better? I'm not sure
> exactly what style we normally use for the diagnostics.

Doesn't this warning trigger even if arm_neon.h is nowhere to be
found? That seems a bit broad, the GNU vector extensions are useful in
their own right, and I'm not sure we should be effectively deprecating
them on ARM.

Unfortunately, I've got no ideas on how to only give a message when
it's relevant. I don't really do much on clang either.

Other than that, you need a Triple::aarch64 check too.

Cheers.

Tim.



More information about the cfe-commits mailing list