[all-commits] [llvm/llvm-project] 882814: [InlineCost] Correct the default branch cost for t...

[Quentin Dian via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 882814edd33cab853859f07b1dd4c4fa1393e0ea
      https://github.com/llvm/llvm-project/commit/882814edd33cab853859f07b1dd4c4fa1393e0ea
  Author: Quentin Dian <dianqk at dianqk.net>
  Date:   2024-05-05 (Sun, 05 May 2024)

  Changed paths:
    M llvm/lib/Analysis/InlineCost.cpp
    A llvm/test/Transforms/Inline/inline-cost-switch-default.ll
    M llvm/test/Transforms/Inline/inline-switch-default-2.ll
    M llvm/test/Transforms/Inline/inline-switch-default.ll

  Log Message:
  -----------
  [InlineCost] Correct the default branch cost for the switch statement (#85160)

Fixes #81723.

The earliest commit of the related code is:
https://github.com/llvm/llvm-project/commit/919f9e8d65ada6552b8b8a5ec12ea49db91c922a.
I tried to understand the following code with
https://github.com/llvm/llvm-project/pull/77856#issuecomment-1993499085.

https://github.com/llvm/llvm-project/blob/5932fcc47855fdd209784f38820422d2369b84b2/llvm/lib/Analysis/InlineCost.cpp#L709-L720

I think only scenarios where there is a default branch were considered.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications