[PATCH] Fix alignment checks in MergeConsecutiveStores.

James Y Knight jyknight at google.com
Fri May 8 08:23:00 PDT 2015


Committed in r236850 and test fixed for darwin in r236855.


================
Comment at: lib/CodeGen/SelectionDAG/DAGCombiner.cpp:10161
@@ +10160,3 @@
+
+static bool AllowableAlignment(const SelectionDAG& DAG,
+                               const TargetLowering& TLI,
----------------
qcolombet wrote:
> 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
It does, fixed.

http://reviews.llvm.org/D8891

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






More information about the llvm-commits mailing list