This is an archive of the discontinued LLVM Phabricator instance.

[mlir][Linalg] Modify `InferStaticShapeOfOperands` to work on Linalg Ops.
ClosedPublic

Authored by mravishankar on Mar 3 2022, 8:33 PM.

Details

Summary

Commit rG1a2bb03edab9d7aa31beb587d0c863acc6715d27 introduced a pattern
to convert dynamic dimensions in operands of GenericOps to static
values based on indexing maps and shapes of other operands. The logic
is directly usable to any LinalgOp. Move that pattern as an
OpInterfaceRewritePattern.

Depends on D120950

Diff Detail

Event Timeline

mravishankar created this revision.Mar 3 2022, 8:33 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 3 2022, 8:33 PM
mravishankar requested review of this revision.Mar 3 2022, 8:33 PM
gysit accepted this revision.Mar 4 2022, 12:17 AM

LGTM modulo the failing test. As mentioned in https://reviews.llvm.org/D120950 this revision seems to have slipped into the base revision somehow.

This revision is now accepted and ready to land.Mar 4 2022, 12:17 AM
gprateek93 accepted this revision.Mar 4 2022, 9:32 AM
okkwon accepted this revision.Mar 4 2022, 11:12 AM

LGTM. Thanks

This revision was landed with ongoing or failed builds.Mar 8 2022, 10:55 AM
This revision was automatically updated to reflect the committed changes.