This is an archive of the discontinued LLVM Phabricator instance.

[mlir][sparse] Improve concatenate operator rewrite for annotated all dense dimensions results.
ClosedPublic

Authored by bixia on Nov 28 2022, 8:13 AM.

Diff Detail

Event Timeline

bixia created this revision.Nov 28 2022, 8:13 AM
Herald added a project: Restricted Project. · View Herald Transcript
bixia requested review of this revision.Nov 28 2022, 8:13 AM
aartbik accepted this revision.Nov 28 2022, 5:27 PM
aartbik added inline comments.
mlir/lib/Dialect/SparseTensor/Transforms/SparseTensorRewriting.cpp
502

maybe make a note in the comment on the all dense case

mlir/test/Dialect/SparseTensor/sparse_concat_codegen.mlir
5

maybe in a next revision, can we also have an example that reorders (like CSC)?

This revision is now accepted and ready to land.Nov 28 2022, 5:27 PM
bixia updated this revision to Diff 478626.Nov 29 2022, 9:20 AM
bixia marked 2 inline comments as done.

Add a test with permutation. Add code comment.

bixia added inline comments.Nov 29 2022, 9:28 AM
mlir/test/Dialect/SparseTensor/sparse_concat_codegen.mlir
5

Add a test for reordering.