[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