diff options
author | Andrzej WarzyĆski <andrzej.warzynski@arm.com> | 2023-11-28 15:02:11 +0000 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-11-28 15:02:11 +0000 |
commit | eb1d5065c560b3468fa0d34af3103359cd78c088 (patch) | |
tree | 0fcdfeeb779b565cdd1329985b1e8575e036d368 /clang/lib/Frontend/ASTMerge.cpp | |
parent | 104b7c624ed54865ea2caf5180ccc8824bddec4b (diff) | |
download | llvm-eb1d5065c560b3468fa0d34af3103359cd78c088.zip llvm-eb1d5065c560b3468fa0d34af3103359cd78c088.tar.gz llvm-eb1d5065c560b3468fa0d34af3103359cd78c088.tar.bz2 |
[MLIR][Vector] Refactor tests for contract -> OP transforms (2/N) (#73445)
This is a direct follow-up of #73348. The matvec trait that's used for
`@matvec_m_mk_k` was incorrectly updated from:
```
#redpar_vecmattrans_accesses = [
affine_map<(m, k) -> (m)>,
affine_map<(m, k) -> (m, k)>,
affine_map<(m, k) -> (k)>
]
indexing_maps = #redpar_vecmattrans_accesses,
iterator_types = ["reduction", "parallel"]
}
```
to:
```
#matvec_accesses_4 = [
affine_map<(m, k) -> (k)>,
affine_map<(m, k) -> (k, m)>,
affine_map<(m, k) -> (m)>
]
indexing_maps = #matvec_accesses_4,
iterator_types = ["parallel", "reduction"]
}
```
Note that these traits describe identical matvec operation, hence the
`CHECK` lines are identical for both.
Also, `#redpar_vecmattrans_trait` is identical to `#matvec_trait_8`
that's already present in:
* "vector-contract-to-outerproduct-matvec-transforms.mlir"
For this reason:
* `@matvec_m_mk_k` is moved near other tests that already use
`#matvec_trait_8`,
* `#redpar_vecmattrans_trait` is replaced `#matvec_trait_8`.
This is a part of a larger effort to add cases with scalable vectors to
tests for the Vector dialect. I am refactoring these tests so that it's
easier to identify what cases are tested and where to add tests for
scalable vectors.
Implements #72834.
Diffstat (limited to 'clang/lib/Frontend/ASTMerge.cpp')
0 files changed, 0 insertions, 0 deletions