This is an archive of the discontinued LLVM Phabricator instance.

[mlir][Linalg] Revisit the Linalg on tensors abstraction
AbandonedPublic

Authored by mravishankar on Dec 21 2020, 9:41 AM.

Details

Summary

[NO NEED TO REVIEW]

This is same as patch https://reviews.llvm.org/D93469

This revision drops init_tensor arguments from Linalg on tensors and instead uniformizes the output buffers and output tensors to be consistent.
This significantly simplifies the usage of Linalg on tensors and is a stepping stone for
its evolution towards a mixed tensor and shape abstraction discussed in https://llvm.discourse.group/t/linalg-and-shapes/2421/19.

Diff Detail

Event Timeline

mravishankar created this revision.Dec 21 2020, 9:41 AM
mravishankar requested review of this revision.Dec 21 2020, 9:41 AM

Is this the same as Nicolas patch?

This is the same as Nicolas' patch. Trying to land it (checking builds). Will drop the reviewers.

mravishankar edited the summary of this revision. (Show Details)Dec 21 2020, 11:43 AM

turns out I cant drop reviewers. Added a note saying no need to review this.

Fix failing tests and builds

This is the same as Nicolas' patch. Trying to land it (checking builds). Will drop the reviewers.

Can't you just land someone's patch without making a new one in phabricator?
That also preserves the authorship.

aartbik added inline comments.Dec 21 2020, 1:38 PM
mlir/lib/Dialect/Linalg/Transforms/Sparsification.cpp
470

the -1 is no longer there!

mravishankar abandoned this revision.Dec 21 2020, 2:02 PM
mlir/lib/Dialect/Linalg/Transforms/Sparsification.cpp
470

Ah sorry! I already landed the patch. Ill approve your patch

mlir/include/mlir/Dialect/Linalg/IR/LinalgStructuredOpsInterface.td