This is an archive of the discontinued LLVM Phabricator instance.

[mlir][sparse] use straightline and loop to insert into tensor
ClosedPublic

Authored by aartbik on Oct 19 2022, 12:45 PM.

Details

Summary

This exposed a missing type conversion for codegen

Diff Detail

Event Timeline

aartbik created this revision.Oct 19 2022, 12:45 PM
Herald added a project: Restricted Project. · View Herald TranscriptOct 19 2022, 12:45 PM
aartbik requested review of this revision.Oct 19 2022, 12:45 PM
aartbik updated this revision to Diff 469015.Oct 19 2022, 12:48 PM

fixed header

aartbik added inline comments.Oct 19 2022, 6:30 PM
mlir/test/Integration/Dialect/SparseTensor/CPU/sparse_insert_1d.mlir
63

Just FYI at the moment. We break here because

(1) bufferization copy
(2) SCF.for does not allow 1:N type conversions yet

Peiming added inline comments.Oct 20 2022, 1:05 PM
mlir/test/Integration/Dialect/SparseTensor/CPU/sparse_insert_1d.mlir
68

You forget a sparse_tensor.load %7 here

aartbik marked an inline comment as done.Oct 20 2022, 1:08 PM
aartbik added inline comments.
mlir/test/Integration/Dialect/SparseTensor/CPU/sparse_insert_1d.mlir
68

I sure did!

aartbik updated this revision to Diff 469333.Oct 20 2022, 1:22 PM
aartbik marked an inline comment as done.

load + hasInserts

aartbik updated this revision to Diff 469684.Oct 21 2022, 10:46 AM

rebased with main, added NOE usage

(note that this of course still needs the 1:N conversion)

aartbik updated this revision to Diff 469784.Oct 21 2022, 3:08 PM

rebased with main, functional now

Peiming accepted this revision.Oct 21 2022, 4:09 PM
This revision is now accepted and ready to land.Oct 21 2022, 4:09 PM
This revision was landed with ongoing or failed builds.Oct 21 2022, 4:17 PM
This revision was automatically updated to reflect the committed changes.