[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