This is an archive of the discontinued LLVM Phabricator instance.

[mlir][sparse] introduce new 2:4 block sparsity level type.
ClosedPublic

Authored by Peiming on Jul 12 2023, 3:19 PM.

Diff Detail

Event Timeline

Peiming created this revision.Jul 12 2023, 3:19 PM
Herald added a project: Restricted Project. · View Herald Transcript
Peiming requested review of this revision.Jul 12 2023, 3:19 PM
aartbik added inline comments.Jul 12 2023, 3:31 PM
mlir/test/Dialect/SparseTensor/roundtrip_encoding.mlir
138

can we use the NEW_SYNTAX for this

aartbik added inline comments.Jul 12 2023, 3:35 PM
mlir/include/mlir/Dialect/SparseTensor/IR/Enums.h
176

use property to encode ...
or
use property for encoding ...

238

no : in syntax, but you can pick

mlir/lib/Bindings/Python/DialectSparseTensor.cpp
22

no :

Peiming updated this revision to Diff 539782.Jul 12 2023, 4:05 PM
Peiming marked 4 inline comments as done.

address comments.

Peiming updated this revision to Diff 539783.Jul 12 2023, 4:07 PM

fix naming inconsistency.

aartbik accepted this revision.Jul 12 2023, 4:10 PM
This revision is now accepted and ready to land.Jul 12 2023, 4:10 PM
This revision was landed with ongoing or failed builds.Jul 12 2023, 4:33 PM
This revision was automatically updated to reflect the committed changes.