[llvm] [InlineCost] Consider the default branch when calculating cost (PR #77856)
Quentin Dian via llvm-commits
llvm-commits at lists.llvm.org
Tue Feb 6 01:03:56 PST 2024
================
@@ -65,7 +65,8 @@ class TargetTransformInfoImplBase {
(void)PSI;
(void)BFI;
JTSize = 0;
- return SI.getNumCases();
+ bool HasDefault = !SI.defaultDestUndefined();
+ return SI.getNumCases() + HasDefault;
----------------
DianQK wrote:
I think it's good semantics for `getNumCases` to not include default branches. I'll update the code to give similar semantics to `getEstimatedNumberOfCaseClusters`. :)
https://github.com/llvm/llvm-project/pull/77856
More information about the llvm-commits
mailing list