This is an archive of the discontinued LLVM Phabricator instance.

[mlir][sparse][taco] Add a few unary operations.
ClosedPublic

Authored by bixia on Mar 10 2022, 9:48 AM.

Details

Summary

Add operations -, abs, ceil and floor to the index notation.

Add test cases.

Diff Detail

Event Timeline

bixia created this revision.Mar 10 2022, 9:48 AM
Herald added a project: Restricted Project. · View Herald Transcript
bixia requested review of this revision.Mar 10 2022, 9:48 AM
aartbik accepted this revision.Mar 10 2022, 11:51 AM
aartbik added inline comments.
mlir/test/Integration/Dialect/SparseTensor/taco/test_tensor_unary_ops.py
20

very neat to see all these new ops!

mlir/test/Integration/Dialect/SparseTensor/taco/tools/mlir_pytaco.py
1631

per our offline discussion, maybe clarify this for people like me that are a bit rusty in Python ;-)
Something like "# Forward decls." or so

This revision is now accepted and ready to land.Mar 10 2022, 11:51 AM
bixia updated this revision to Diff 414496.Mar 10 2022, 2:06 PM

Changed ... to pass and added comments.

mlir/test/Integration/Dialect/SparseTensor/taco/test_tensor_unary_ops.py
20

Thanks!

mlir/test/Integration/Dialect/SparseTensor/taco/tools/mlir_pytaco.py
1631

I replaced ... with pass and added comments.

bixia updated this revision to Diff 414515.Mar 10 2022, 3:32 PM

Rebased.

This revision was automatically updated to reflect the committed changes.