[all-commits] [llvm/llvm-project] 971444: [mlir][linalg][bufferize] Remove redundant methods...

Matthias Springer via All-commits all-commits at lists.llvm.org
Thu Nov 4 17:44:25 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 9714444f1e43fafc7e5235f4d3675b9d2056d97b
      https://github.com/llvm/llvm-project/commit/9714444f1e43fafc7e5235f4d3675b9d2056d97b
  Author: Matthias Springer <springerm at google.com>
  Date:   2021-11-05 (Fri, 05 Nov 2021)

  Changed paths:
    M mlir/include/mlir/Dialect/Linalg/Transforms/BufferizableOpInterface.td
    M mlir/lib/Dialect/Linalg/Transforms/ComprehensiveBufferize.cpp

  Log Message:
  -----------
  [mlir][linalg][bufferize] Remove redundant methods from op interface

These two methods are redundant and removed:
* `bufferizesToAliasOnly`: If not `bufferizesToMemoryRead` and not `bufferizesToMemoryWrite` but `getAliasingOpResult` returns a non-null value, we know that this OpOperand is alias-only. This method now has a default implementation and does not have to be implemented.
* `getInplaceableOpResult`: The analysis does not differentiate between "inplaceable" and "aliasing". The only thing that matters is whether or not OpOperand and OpResult are aliasing. That is the key property that makes buffer copies necessary.

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




More information about the All-commits mailing list