[all-commits] [llvm/llvm-project] 26821f: [mlir][NFC] accept plain OpBuidler in folded const...

ftynse via All-commits all-commits at lists.llvm.org
Fri Jul 29 09:02:09 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 26821f75ed572735c9e2544903b62b47ffde359d
      https://github.com/llvm/llvm-project/commit/26821f75ed572735c9e2544903b62b47ffde359d
  Author: Alex Zinenko <zinenko at google.com>
  Date:   2022-07-29 (Fri, 29 Jul 2022)

  Changed paths:
    M mlir/include/mlir/Dialect/Affine/IR/AffineOps.h
    M mlir/lib/Dialect/Affine/IR/AffineOps.cpp
    M mlir/lib/Dialect/Linalg/Transforms/Tiling.cpp
    M mlir/lib/Dialect/Linalg/Transforms/TilingInterfaceImpl.cpp

  Log Message:
  -----------
  [mlir][NFC] accept plain OpBuidler in folded construction helpers

A group of functions in the Affine dialect provides a mechanism for
buliding folded-by-construction operations. These functions used to
accept a `RewriterBase` reference because they may need to erase the
operations that were folded and notify the rewriter when called from
rewrite patterns. Adopt a different approach: postpone the builder
notification of the op creation until we are certain that the op will
not be folded away. This removes the need to notify the rewriter about
op deletion following op construction in case of successful folding, and
removes a bunch of one-off `IRRewriter` instances in transform code that
may mess up insertion points.

Reviewed By: springerm, mravishankar

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




More information about the All-commits mailing list