[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