[PATCH] D136377: [mlir][MemRefToLLVM] Reuse existing subview lowering

[Alex Zinenko via Phabricator via llvm-commits]

ftynse added a comment.

I am really not a fan of the layering here. This goes back to what we had with uber-passes "convert-X-Y-Z-to-A-B-C" with big dependencies. It is very surprising that running `convert-memref-to-llvm` will also convert any affine operations directly to LLVM, instead of lowering them to Arith where they can be jointly optimized with other arithmetic operations.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136377