[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