<br><br><div class="gmail_quote">On Fri Dec 05 2014 at 3:56:09 PM Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Oliver,<div><br></div><div>I'm sorry for the thread archaeology, but I didn't see this when it went in.</div><div><br><div class="gmail_quote"></div></div><div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
[ARM] Change default ABI for AArch32 to be "aapcs" (was "apcs-gnu")<br>
<br>
The current default abi when no environment is given is "apcs-gnu",<br>
which is obsolete. This patch changes the default to "aapcs". "aapcs" has both<br>
hard- and soft-float variants, so the -mhard-float, -msoft-float and<br>
-mfloat-abi= options now all behave as expected when no environment is<br>
specified in the triple.<br>
<br></blockquote><div><br></div></div></div><div><div class="gmail_quote"><div>Is there an actual compelling reason to make this change? Does it get us conformity with any other tools? </div><div><br></div><div>As a note, you also didn't change the backend to match this which is causing me some amount of annoyance in trying to get everything to agree. I'm unraveling it, but it's taking some time. :)</div></div></div><div><div class="gmail_quote"><div> </div></div></div></blockquote><div><br></div><div>Ping? Do you have a reason for this change? Basically it means a lot of updating the back end if we want them to agree for no good reason so I'm of mind to revert this.</div><div><br></div><div>-eric</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
While writing this I also noticed that a preprocessor test claims to be<br>
checking darwin, but is actually checking the defaults, which are<br>
different for darwin.<br>
<br></blockquote><div><br></div></div></div><div><div class="gmail_quote"><div>Weird.</div></div></div><div><div class="gmail_quote"><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
-// RUN: %clang -target arm -arch armv7s -x c -E -dM %s -o - | FileCheck --check-prefix=ARM-DARWIN-NO-<u></u>E<u></u>ABI %s<br>
-// RUN: %clang -target arm -arch armv6m -x c -E -dM %s -o - | FileCheck --check-prefix=ARM-DARWIN-NO-<u></u>E<u></u>ABI %s<br>
-// RUN: %clang -target arm -arch armv7m -x c -E -dM %s -o - | FileCheck --check-prefix=ARM-DARWIN-NO-<u></u>E<u></u>ABI %s<br>
-// RUN: %clang -target arm -arch armv7em -x c -E -dM %s -o - | FileCheck --check-prefix=ARM-DARWIN-NO-<u></u>E<u></u>ABI %s<br>
+// RUN: %clang -target arm-apple-darwin-eabi -arch armv7s -x c -E -dM %s -o - | FileCheck --check-prefix=ARM-DARWIN-NO-<u></u>E<u></u>ABI %s<br>
+// RUN: %clang -target arm-apple-darwin-eabi -arch armv6m -x c -E -dM %s -o - | FileCheck --check-prefix=ARM-DARWIN-EABI %s<br>
+// RUN: %clang -target arm-apple-darwin-eabi -arch armv7m -x c -E -dM %s -o - | FileCheck --check-prefix=ARM-DARWIN-EABI %s<br>
+// RUN: %clang -target arm-apple-darwin-eabi -arch armv7em -x c -E -dM %s -o - | FileCheck --check-prefix=ARM-DARWIN-EABI %s<br>
 // RUN: %clang -target thumbv7-apple-darwin-eabi -arch armv7 -x c -E -dM %s -o - | FileCheck --check-prefix=ARM-DARWIN-NO-<u></u>E<u></u>ABI %s<br>
 // ARM-DARWIN-NO-EABI-NOT: #define __ARM_EABI__ 1<br>
+// ARM-DARWIN-EABI: #define __ARM_EABI__ 1<br><br></blockquote><div><br></div></div></div><div><div class="gmail_quote"><div>I don't think this change is quite correct, it would be better to use a triple that acutally exists and then maybe -mabi instead? cc'ing Tim as he put that part in originally.</div></div></div><div><div class="gmail_quote"><div><br></div><div>-eric</div></div></div></blockquote></div>