[llvm] TTI: Fix special casing vectorization costs of saturating add/sub (PR #97463)

David Green via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 24 23:16:42 PDT 2024


https://github.com/davemgreen commented:

The `llvm.sadd.sat.i64` Arm costs are a bit odd - I assume because `getTypeLegalizationCost(i64)` will be `{2, i32}`, and it uses the `LT.first * 2` as the `i32` is legal. The other results seem OK but that one doesn't sound right for this operation. I could add an override in the Arm backend if you think that is the best way forward - the other results look better, including the SVE versions that we don't currently have tests for.

https://github.com/llvm/llvm-project/pull/97463


More information about the llvm-commits mailing list