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

Eric Christopher via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 30 16:40:35 PDT 2016


echristo added a comment.

Couple of inline comments. I think some of the overall design can be separated out and be made a bit simpler - especially as relates to ForcedLayoutPred etc, but we can worry about that as a cleanup that'll happen next.

Thanks!

-eric


================
Comment at: lib/CodeGen/BranchFolding.cpp:641
@@ -638,2 +640,3 @@
   // heuristics.
+  // This is broken, as it breaks tail-dup then merge.
   unsigned EffectiveTailLen = CommonTailLen;
----------------
Can you elaborate here? Perhaps with a FIXME or a TODO.

================
Comment at: lib/CodeGen/MachineBlockPlacement.cpp:1696
@@ +1695,3 @@
+  if (TailDupPlacement)
+    TailDup.initMF(MF, MMI, MBPI);
+
----------------
The MachineFunction has an MMI so you don't need this argument. :)


http://reviews.llvm.org/D18226





More information about the llvm-commits mailing list