[PATCH] [ARM] Use AEABI aligned function variants

John Brawn john.brawn at arm.com
Wed Apr 15 03:41:44 PDT 2015


REPOSITORY
  rL LLVM

================
Comment at: lib/Target/ARM/ARMSelectionDAGInfo.cpp:36
@@ +35,3 @@
+                       RTLIB::Libcall LC) const
+{
+  const ARMSubtarget &Subtarget =
----------------
asl wrote:
> Minor nit: put brace on the preceding line
Will do.

================
Comment at: lib/Target/ARM/ARMSelectionDAGInfo.cpp:43
@@ +42,3 @@
+  // Libcall is an AEABI function.
+  if (std::strncmp(TLI->getLibcallName(LC), "__aeabi", 7) != 0)
+    return SDValue();
----------------
asl wrote:
> Should we start with cheaper ABI check here and bail out early for non-AAPCS?
I'm not sure what you mean by this?

http://reviews.llvm.org/D8060

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list