[llvm] [RISCV] Remove setJumpIsExpensive(). (PR #74647)
Alex Bradbury via llvm-commits
llvm-commits at lists.llvm.org
Thu Dec 7 07:11:30 PST 2023
asb wrote:
> How about adding it to `RISCVSubtarget` and changing it to something like:
>
> ```c
> setJumpIsExpensive(Subtarget.isJumpExpensive())
> ```
>
> So that processors can tune it.
Do you have a target where you actively want to support that configuration? I agree this is something that could be made configurable, but we're probably best off focusing our efforts around a smaller set of configuration options unless we know there's a case where we really need to diverge.
https://github.com/llvm/llvm-project/pull/74647
More information about the llvm-commits
mailing list