[PATCH] D22414: [BranchProbabilityInfo] Change analysis result storage to simplify basic block removal

Igor Laevsky via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 20 09:51:29 PDT 2016


igor-laevsky added a comment.

Hi,

I don't have specific build times. This is mostly motivated by the review comments in the https://reviews.llvm.org/D20957. Evidence for importance of this is that LVI also had linear time EraseBlock and it caused problems there. BPI will call EraseBlock exactly as frequently as LVI did, so it's natural to assume that performance bottleneck will be in the same place.


https://reviews.llvm.org/D22414





More information about the llvm-commits mailing list