[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