[PATCH] D136377: [mlir][MemRefToLLVM] Remove the code for lowering subview

[Quentin Colombet via Phabricator via llvm-commits]

qcolombet added inline comments.


================
Comment at: mlir/test/Conversion/MemRefToLLVM/expand-then-convert-to-llvm.mlir:8-13
+// Note: We run CSE in that test to get rid of duplicated
+// unrealized_conversion_cast that are inserted with
+// convert-memref-to-llvm and then convert-arith-to-llvm.
+// The final code is still not perfect, because we have
+// noop unrealized_conversion_cast from i64 to index
+// and back.
----------------
ftynse wrote:
> I'd expect `-reconcile-unrealized-casts` to fix that.
Ah thanks, let me add that to the pipeline.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136377