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"><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>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><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>Weird.</div><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>EABI %s<br>
-// RUN: %clang -target arm -arch armv6m -x c -E -dM %s -o - | FileCheck --check-prefix=ARM-DARWIN-NO-<u></u>EABI %s<br>
-// RUN: %clang -target arm -arch armv7m -x c -E -dM %s -o - | FileCheck --check-prefix=ARM-DARWIN-NO-<u></u>EABI %s<br>
-// RUN: %clang -target arm -arch armv7em -x c -E -dM %s -o - | FileCheck --check-prefix=ARM-DARWIN-NO-<u></u>EABI %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>EABI %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>EABI %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>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><br></div><div>-eric</div></div></div>