[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