[PATCH] D18077: Factor out MachineBlockPlacement::fillWorkLists. NFC

Amaury SECHET via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 11 12:36:51 PST 2016


deadalnix added inline comments.

================
Comment at: lib/CodeGen/MachineBlockPlacement.cpp:598
@@ +597,3 @@
+
+  if (Chain.UnscheduledPredecessors != 0)
+    return;
----------------
mcrosier wrote:
> Was this style suggested in D17625?
D17625 introduce 2 worklist, so you need to add code there to choose which worklist you add the MBB to.


http://reviews.llvm.org/D18077





More information about the llvm-commits mailing list