[all-commits] [llvm/llvm-project] b3f611: [mlir][Linalg] Replace SimplePad with PadTensor in...

Han-Chung Wang via All-commits all-commits at lists.llvm.org
Mon Feb 1 11:39:39 PST 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: b3f611bfe7d4aac7b1393ab0200d7166a590791b
      https://github.com/llvm/llvm-project/commit/b3f611bfe7d4aac7b1393ab0200d7166a590791b
  Author: Hanhan Wang <hanchung at google.com>
  Date:   2021-02-01 (Mon, 01 Feb 2021)

  Changed paths:
    M mlir/include/mlir/Dialect/Linalg/IR/LinalgOps.td
    M mlir/include/mlir/Dialect/Linalg/Transforms/Hoisting.h
    M mlir/lib/Analysis/CMakeLists.txt
    M mlir/lib/Analysis/SliceAnalysis.cpp
    M mlir/lib/Dialect/Linalg/Transforms/Hoisting.cpp
    M mlir/test/Dialect/Linalg/hoist-padding.mlir
    M mlir/test/Dialect/Linalg/roundtrip.mlir
    M mlir/test/lib/Transforms/TestLinalgTransforms.cpp

  Log Message:
  -----------
  [mlir][Linalg] Replace SimplePad with PadTensor in hoist-padding

This is the last revision to migrate using SimplePadOp to PadTensorOp, and the
SimplePadOp is removed in the patch. Update a bit in SliceAnalysis because the
PadTensorOp takes a region different from SimplePadOp. This is not covered by
LinalgOp because it is not a structured op.

Also, remove a duplicated comment from cpp file, which is already described in a
header file. And update the pseudo-mlir in the comment.

This is as same as D95615 but fixing one dep in CMakeLists.txt

Different from D95671, the fix was applied to run target.

Reviewed By: mravishankar

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




More information about the All-commits mailing list