[PATCH] D22669: [MBP] Added some more debug messages and some clean ups /NFC

David Li via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 22 11:17:15 PDT 2016


davidxl accepted this revision.
davidxl added a comment.
This revision is now accepted and ready to land.

Otherwise looks good.


================
Comment at: lib/CodeGen/MachineBlockPlacement.cpp:570
@@ +569,3 @@
+  //
+  // Here, we are evaluating to select Succ or Pred as a successor of BB.
+  // If we choose Succ, then we break the topological ordering and Pred needs
----------------
I suggest removing change in this paragraph -- it does not seem to make things clearer.


https://reviews.llvm.org/D22669





More information about the llvm-commits mailing list