[all-commits] [llvm/llvm-project] 6700a2: [mlir][linalg][bufferize] Fix insertion point Init...

[Matthias Springer via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 6700a26d5f349c05a38d47a555ba2b24b5b3fcec
      https://github.com/llvm/llvm-project/commit/6700a26d5f349c05a38d47a555ba2b24b5b3fcec
  Author: Matthias Springer <springerm at google.com>
  Date:   2022-01-30 (Sun, 30 Jan 2022)

  Changed paths:
    M mlir/include/mlir/Dialect/Linalg/ComprehensiveBufferize/LinalgInterfaceImpl.h
    M mlir/lib/Dialect/Linalg/ComprehensiveBufferize/LinalgInterfaceImpl.cpp
    M mlir/test/Dialect/Linalg/comprehensive-module-bufferize-init-tensor-elimination.mlir

  Log Message:
  -----------
  [mlir][linalg][bufferize] Fix insertion point InitTensorElimination

There was a bug where some of the OpOperands needed in the replacement op were not in scope.

It does not matter where the replacement op is inserted. Any insertion point is OK as long as there are no dominance errors. In the worst case, the newly inserted op will bufferize out-of-place. This is no worse than not eliminating the InitTensorOp at all.

Differential Revision: https://reviews.llvm.org/D117685