[PATCH] D32529: [GlobalISel] Make multi-step legalization work.

Tim Northover via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 29 12:15:42 PDT 2017


t.p.northover accepted this revision.
t.p.northover added a comment.
This revision is now accepted and ready to land.

Looks fine to me. Just one tiny nit (sorry).



================
Comment at: include/llvm/CodeGen/GlobalISel/LegalizerInfo.h:110
 
+  static bool NeedsLegalizingToDifferentSize(const LegalizeAction Action) {
+    switch (Action) {
----------------
Functions should start with lower-case letters unless they need to match surrounding code.


https://reviews.llvm.org/D32529





More information about the llvm-commits mailing list