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

Kristof Beyls via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 30 01:25:57 PDT 2017


kristof.beyls marked an inline comment as done.
kristof.beyls added inline comments.


================
Comment at: include/llvm/CodeGen/GlobalISel/LegalizerInfo.h:110
 
+  static bool NeedsLegalizingToDifferentSize(const LegalizeAction Action) {
+    switch (Action) {
----------------
t.p.northover wrote:
> Functions should start with lower-case letters unless they need to match surrounding code.
D'oh! Of course.

Thanks for the review!


https://reviews.llvm.org/D32529





More information about the llvm-commits mailing list