[llvm] [InstCombine] Abandon BranchProbabilityAnalysis cache result (PR #86470)

via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 27 23:57:43 PDT 2024


WenleiHe wrote:

> > The change InstCombine does to the CFG is trivial -- can we update the analysis instead?
> 
> I'm afraid that's not easy... We many need to check & update cached BranchProbabilityAnalysis in all callsite of BranchInst.swapSuccessors() or similar operation on SwitchInst, JumpTable...

Can we use `swapSuccEdgesProbabilities` of BPI to update for inverted branches? There aren't many callsites for `swapSuccessors`

https://github.com/llvm/llvm-project/pull/86470


More information about the llvm-commits mailing list