This is an archive of the discontinued LLVM Phabricator instance.

[mlir][sparse] codegen for sparse alloc
ClosedPublic

Authored by aartbik on Sep 2 2022, 6:08 PM.

Diff Detail

Event Timeline

aartbik created this revision.Sep 2 2022, 6:08 PM
Herald added a project: Restricted Project. · View Herald TranscriptSep 2 2022, 6:08 PM
aartbik requested review of this revision.Sep 2 2022, 6:08 PM
Peiming added inline comments.Sep 5 2022, 1:56 PM
mlir/lib/Dialect/SparseTensor/Transforms/SparseTensorCodegen.cpp
177

should I change the class name as well?

aartbik added inline comments.Sep 6 2022, 9:01 AM
mlir/lib/Dialect/SparseTensor/Transforms/SparseTensorCodegen.cpp
177

Yeah, probably a good idea to remove the "New" from this op altogether

Peiming accepted this revision.Sep 6 2022, 9:24 AM
Peiming added inline comments.
mlir/lib/Dialect/SparseTensor/Transforms/SparseTensorCodegen.cpp
177

Okay

This revision is now accepted and ready to land.Sep 6 2022, 9:24 AM
This revision was automatically updated to reflect the committed changes.