<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Aug 6, 2015 at 4:49 PM, Ben Langmuir <span dir="ltr"><<a href="mailto:blangmuir@apple.com" target="_blank">blangmuir@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">benlangmuir added inline comments.<br>
<span class=""><br>
================<br>
Comment at: lib/Basic/Targets.cpp:4552<br>
@@ -4551,2 +4551,3 @@<br>
         .Case("arm", true)<br>
+        .Case("aarch32", true)<br>
         .Case("softfloat", SoftFloat)<br>
----------------<br>
</span><span class="">rsmith wrote:<br>
> Should we also recognize "arm32" to better mirror the 64-bit case?<br>
</span>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).</blockquote><div><br></div><div>OK, SGTM </div></div></div></div>