[PATCH] D73656: [mlir][Linalg] Add a Linalg DRR test to go from matmul to vectors

[River Riddle via Phabricator via llvm-commits]

rriddle added inline comments.


================
Comment at: mlir/test/lib/Transforms/TestLinalgMatmulToVector.cpp:35
+    // TODO(ntv, rriddle) Need ViewOp canonicalizer to enable vectorization
+    // without phase ordering. Unfortunately there is some pattern interaction
+    // because ViewOp and MemRefCastOp canonicalization order need to happen
----------------
This sounds like a visitation ordering problem, i.e. you want to visit the source operand(AffineApplyOp) before the user(ViewOp). Greedy walks reverse order so you run into the problem here. Am I understanding this correctly?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73656