- Fixed vector instructions costs estimations incosistency - removed different logic for "not simple types" since it biases costs for these types.
- Fixed legalization penalty for vectors too big for the target: changed from overwrite default legalization cost value estimation to added penalty.
- Fixed few typos in tests.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
llvm/test/Analysis/CostModel/AMDGPU/add-sub.ll | ||
---|---|---|
75 | Why dropping this? |
llvm/test/Analysis/CostModel/AMDGPU/add-sub.ll | ||
---|---|---|
75 | Actually tests for long vectors are needed to check the penalty is added in getTypeLegalizationCost(). But it is about vectors sized >256bits so we just need to check that cost for v5i64 is not a linear dependency with v4i64 and smaller vectors. |
Why dropping this?