[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