[PATCH] D11824: Make target feature 'arm' cover both 32 and 64 bit architecutres

[Richard Smith via cfe-commits]

On Thu, Aug 6, 2015 at 4:49 PM, Ben Langmuir <blangmuir at apple.com> wrote:

> benlangmuir added inline comments.
>
> ================
> Comment at: lib/Basic/Targets.cpp:4552
> @@ -4551,2 +4551,3 @@
>          .Case("arm", true)
> +        .Case("aarch32", true)
>          .Case("softfloat", SoftFloat)
> ----------------
> rsmith wrote:
> > Should we also recognize "arm32" to better mirror the 64-bit case?
> Kristof convinced me it was better to stick with aarch32 since it had a
> defined meaning unlike arm32 (and unlike arm64 there is no compatibility or
> historical reason for it).


OK, SGTM
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150806/d470a069/attachment.html>