[PATCH] D22892: Codegen: MachineBlockPlacement Improve probability layout.

Kyle Butt via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 28 14:56:09 PDT 2016


iteratee added inline comments.

================
Comment at: lib/CodeGen/MachineBlockPlacement.cpp:661
@@ +660,3 @@
+  // In the worst case, S2 will not get layed out after Pred.
+  // non-topo-cost = 2 * freq(S->Pred) + freq(BB->S2).
+  // To be conservative, we can assume that min(freq(Pred->S1), freq(Pred->S2))
----------------
I think I'll stick with cost, as that's how the other 2 cases are explained.


https://reviews.llvm.org/D22892





More information about the llvm-commits mailing list