[PATCH] D22521: [MBP] do not reorder and move up loop latch block

Mandeep Singh Grang via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 19 10:48:12 PDT 2016


mgrang added a subscriber: mgrang.

================
Comment at: lib/CodeGen/MachineBlockPlacement.cpp:1112
@@ -1070,2 +1111,3 @@
     return nullptr;
+  }
 
----------------
Can we change the two ifs into an if-else clause and move the "return nullptr" outside the if-else? (and also get rid of the braces after the ifs). This will make the code more compact and reduce duplication.


https://reviews.llvm.org/D22521





More information about the llvm-commits mailing list