This is an archive of the discontinued LLVM Phabricator instance.

[mlir][sparse] refactor sparse compiler pipeline to single place
ClosedPublic

Authored by aartbik on Feb 22 2022, 12:30 PM.

Diff Detail

Event Timeline

aartbik created this revision.Feb 22 2022, 12:30 PM
aartbik requested review of this revision.Feb 22 2022, 12:30 PM
aartbik updated this revision to Diff 410624.Feb 22 2022, 1:06 PM

rebased with main head

bixia accepted this revision.Feb 22 2022, 2:16 PM

Nice!

mlir/test/Integration/Dialect/SparseTensor/python/tools/sparse_compiler.py
7

Shall we write this as
from mlir import all_passes_registration
so it is consistent with the following two lines?

If we will make this change, we can also move this line to the same block with the following two lines.

This revision is now accepted and ready to land.Feb 22 2022, 2:16 PM
aartbik marked an inline comment as done.Feb 22 2022, 4:01 PM
aartbik added inline comments.
mlir/test/Integration/Dialect/SparseTensor/python/tools/sparse_compiler.py
7

Yes, looks nicer. Done.

aartbik updated this revision to Diff 410664.Feb 22 2022, 4:02 PM
aartbik marked an inline comment as done.

addressed comment