[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