[PATCH] D72935: [mlir] Add a canonicalization pattern for MemRefCastOp into dynamic MemRefs

[Nicolas Vasilache via Phabricator via llvm-commits]

nicolasvasilache abandoned this revision.
nicolasvasilache added a comment.

Scratch that, I am wrong, sorry .. the load(memref) just forwards the operand and that is ok.
It seems my current pattern is not properly testable without the ops I want.

I will move it into Linalg Transforms for now and get the end-to-end story more visible. 
When/if it makes sense we can discuss whether canonicalization could ever make sense for this.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D72935/new/

https://reviews.llvm.org/D72935