[llvm] TTI: Fix special casing vectorization costs of saturating add/sub (PR #97463)
Matt Arsenault via llvm-commits
llvm-commits at lists.llvm.org
Wed Jul 3 02:01:06 PDT 2024
arsenm wrote:
> or any backends, such as AArch64, which have already made an attempt to cost the saturating intrinsics, it looks like we're now losing out.
How is this attempting to cost these? This was just adding up the cost of the assumed expansion. I don't think any of this code here adding up a hypothetical expansions is reasonable to have as a default
>
> I'm wondering if using `getArithmeticInstrCost` would help in the general sense even though, from a brief look, the AArch64 backend wouldn't handle it properly there either.
I have no idea what's going on with the soup of TTI cost functions. It's like there are 3-10 competing APIs intermixed
https://github.com/llvm/llvm-project/pull/97463
More information about the llvm-commits
mailing list