This is an archive of the discontinued LLVM Phabricator instance.

[mlir][sparse] extend foreach operation to iterator over sparse constant.
ClosedPublic

Authored by Peiming on Nov 8 2022, 4:43 PM.

Diff Detail

Event Timeline

Peiming created this revision.Nov 8 2022, 4:43 PM
Herald added a project: Restricted Project. · View Herald Transcript
Peiming requested review of this revision.Nov 8 2022, 4:43 PM
aartbik accepted this revision.Nov 8 2022, 5:00 PM
aartbik added inline comments.
mlir/lib/Dialect/SparseTensor/Transforms/SparseTensorRewriting.cpp
703

I would add a comment before L703

// Special-case: for each over a sparse constant uses its own rewriting rule.

Just to set it apart from the 1.2. steps below

also, perhaps later, perhaps we should move some of these larger codeblocks into their own helper method

This revision is now accepted and ready to land.Nov 8 2022, 5:00 PM
Peiming updated this revision to Diff 474128.Nov 8 2022, 5:48 PM
Peiming marked an inline comment as done.

address comments.

This revision was landed with ongoing or failed builds.Nov 8 2022, 5:50 PM
This revision was automatically updated to reflect the committed changes.