Add cost model tests for upcoming dot product lowering change. Related because we had to fix the arm64 cost model for our change.
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
Thanks! Could you also add tests for reductions with just the reassoicate fast-math flag?
Also, this is not really specific to matrix support in LLVM, but applies generally to the AArch64 cost modeling. So it would be better to use [AArch64] in the description.
llvm/test/Analysis/CostModel/AArch64/reduce-fadd.ll | ||
---|---|---|
25 | I think the strict part here refers to the ordering requirement, i.e. strict sequential order. So it would probably be better to name the function @fast_fp_reductions. | |
40 | can you also throw in some vectors with some odd numbers of elements. |
Comment Actions
Replaced with the following review: https://reviews.llvm.org/D127682. Squashing prior to running arc diff created a new review instead of updating this one.
I think the strict part here refers to the ordering requirement, i.e. strict sequential order.
So it would probably be better to name the function @fast_fp_reductions.