[PATCH] Fix alignment checks in MergeConsecutiveStores.
Quentin Colombet
qcolombet at apple.com
Tue May 5 14:26:21 PDT 2015
Hi James,
LGTM.
Thanks,
-Quentin
================
Comment at: lib/CodeGen/SelectionDAG/DAGCombiner.cpp:10161
@@ +10160,3 @@
+
+static bool AllowableAlignment(const SelectionDAG& DAG,
+ const TargetLowering& TLI,
----------------
I think the coding style says lower case for the first letter of a function name.
http://llvm.org/docs/CodingStandards.html#name-types-functions-variables-and-enumerators-properly
http://reviews.llvm.org/D8891
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list