[PATCH] D142924: [llvm][IfConversion] update successor list when merging INLINEASM_BR

[Bill Wendling via Phabricator via llvm-commits]

void accepted this revision.
void added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/lib/CodeGen/IfConversion.cpp:2254
+          if (MO.isMBB() && !ToBBI.BB->isSuccessor(MO.getMBB()))
+            ToBBI.BB->addSuccessor(MO.getMBB(), BranchProbability::getZero());
+
----------------
void wrote:
> nickdesaulniers wrote:
> > nickdesaulniers wrote:
> > > Perhaps it might be best to see if the prior branch had a branch probability, and if so, reuse that?
> > > 
> > > 
> > > See calls to `getEdgeProbability` below.
> > eh maybe not. Pretty sure we only create these edges with 0% probability. The fallthough BB is not an explicit operand.
> +1. I think we mark all indirect edges as "cold" by default.
Do we already move the indirect blocks to the end of the function? Or would that be too pessimistic?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142924