[PATCH] D78806: SplitIndirectBrCriticalEdges: Fix Branch Probability update

Hiroshi Yamauchi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 24 10:16:12 PDT 2020


yamauchi added a comment.

Why considering the cases where "target basic block happens to have unset a probability for any of its successors" specifically?

Does the new test TEST(BasicBlockUtils, SplitIndirectBrCriticalEdge) fail before the change in BreakCriticalEdges.cpp (where the target block doesn't seem to have an unset successor)?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D78806/new/

https://reviews.llvm.org/D78806





More information about the llvm-commits mailing list