[PATCH] [ARM] Use AEABI aligned function variants

Jonathan Roelofs jonathan at codesourcery.com
Fri May 8 11:08:24 PDT 2015


REPOSITORY
  rL LLVM

================
Comment at: lib/Target/ARM/ARMSelectionDAGInfo.cpp:78
@@ +77,3 @@
+  if ((Align & 7) == 0) {
+    AlignVariant = ALIGN8;
+  } else if ((Align & 3) == 0) {
----------------
No need for '{'s on single-statement if/else-if/else blocks.

================
Comment at: lib/Target/ARM/ARMSelectionDAGInfo.cpp:104
@@ +103,3 @@
+    else
+      Src = DAG.getNode(ISD::ZERO_EXTEND, dl, MVT::i32, Src);
+
----------------
If it's already MVT::i32, don't generate the zext?

http://reviews.llvm.org/D8060

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






More information about the llvm-commits mailing list