[PATCH] D18226: Codegen: Tail-duplicate during placement.

[David Li via llvm-commits]

davidxl added inline comments.

================
Comment at: lib/CodeGen/MachineBlockPlacement.cpp:1862
@@ +1861,3 @@
+/// \p DuplicatedToLPred - True if the block was duplicated into LPred. Will
+///                        only be true if Removed is also true.
+void MachineBlockPlacement::maybeTailDuplicateBlock(
----------------
Is this comment correct? The TailBB may be dup'ed in to Lpred, but still not removed as there are some other predecessors it can not be dup'ed into ?

================
Comment at: lib/CodeGen/MachineBlockPlacement.cpp:1951
@@ +1950,3 @@
+
+  DuplicatedToLPred = Removed && CanDupToChain;
+}
----------------
Should DuplicatedToLPred == CanDupToChain?


https://reviews.llvm.org/D18226