[all-commits] [llvm/llvm-project] d7eba2: [mlir][Inliner] Refactor the inliner to use nested...

River Riddle via All-commits all-commits at lists.llvm.org
Mon Dec 14 18:28:38 PST 2020


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: d7eba2005267aa4a8f46f73f208c7cc23e6c6a1a
      https://github.com/llvm/llvm-project/commit/d7eba2005267aa4a8f46f73f208c7cc23e6c6a1a
  Author: River Riddle <riddleriver at gmail.com>
  Date:   2020-12-14 (Mon, 14 Dec 2020)

  Changed paths:
    M llvm/include/llvm/ADT/Sequence.h
    M mlir/include/mlir/Pass/AnalysisManager.h
    M mlir/include/mlir/Pass/Pass.h
    M mlir/include/mlir/Pass/PassManager.h
    M mlir/include/mlir/Transforms/Passes.h
    M mlir/include/mlir/Transforms/Passes.td
    M mlir/lib/Pass/Pass.cpp
    M mlir/lib/Pass/PassDetail.h
    M mlir/lib/Pass/PassRegistry.cpp
    M mlir/lib/Pass/PassTiming.cpp
    M mlir/lib/Transforms/Inliner.cpp
    M mlir/test/Dialect/Affine/inlining.mlir
    M mlir/test/Dialect/SPIRV/Transforms/inlining.mlir
    M mlir/test/Pass/dynamic-pipeline-nested.mlir
    M mlir/test/Transforms/inlining.mlir
    M mlir/test/lib/Transforms/TestDynamicPipeline.cpp

  Log Message:
  -----------
  [mlir][Inliner] Refactor the inliner to use nested pass pipelines instead of just canonicalization

Now that passes have support for running nested pipelines, the inliner can now allow for users to provide proper nested pipelines to use for optimization during inlining. This revision also changes the behavior of optimization during inlining to optimize before attempting to inline, which should lead to a more accurate cost model and prevents the need for users to schedule additional duplicate cleanup passes before/after the inliner that would already be run during inlining.

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




More information about the All-commits mailing list