[PATCH] [ARM] Use AEABI aligned function variants

Anton Korobeynikov anton at korobeynikov.info
Tue Apr 14 06:15:40 PDT 2015


Otherwise LGTM. Renato, what do you think?


REPOSITORY
  rL LLVM

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

================
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();
----------------
Should we start with cheaper ABI check here and bail out early for non-AAPCS?

http://reviews.llvm.org/D8060

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






More information about the llvm-commits mailing list