This is an archive of the discontinued LLVM Phabricator instance.

[mlir][memref] Add memref.copy operation
ClosedPublic

Authored by herhut on Jun 21 2021, 10:34 AM.

Details

Summary

As the name suggests, it copies from one memref to another.

Diff Detail

Event Timeline

herhut created this revision.Jun 21 2021, 10:34 AM
herhut requested review of this revision.Jun 21 2021, 10:34 AM
pifon2a accepted this revision.Jun 21 2021, 10:39 AM
pifon2a added inline comments.
mlir/include/mlir/Dialect/MemRef/IR/MemRefOps.td
452

does it mean SameOperandsType, although we don't have this trait in mlir yet? Are different layouts allowed?

This revision is now accepted and ready to land.Jun 21 2021, 10:39 AM
herhut updated this revision to Diff 353603.Jun 22 2021, 4:19 AM

Added more detail in comment

herhut added inline comments.Jun 22 2021, 4:21 AM
mlir/include/mlir/Dialect/MemRef/IR/MemRefOps.td
452

I was not sure whether SameOperandsType would imply that the types have to be identical, which they don't. They may differ in the used maps and I have extended the comment a little.

This revision was landed with ongoing or failed builds.Jun 22 2021, 4:22 AM
This revision was automatically updated to reflect the committed changes.