[PATCH] D91211: [mlir][Inliner] Refactor the inliner to use nested pass pipelines instead of just canonicalization

[River Riddle via Phabricator via llvm-commits]

rriddle created this revision.
rriddle added a reviewer: mehdi_amini.
Herald added subscribers: llvm-commits, dexonsmith, rdzhabarov, tatianashp, ThomasRaoux, AlexeySotkin, msifontes, jurahul, Kayjukh, grosul1, Joonsoo, liufengdb, aartbik, lucyrfox, mgester, arpith-jacob, antiagainst, shauheen, jfb, eraman.
Herald added a reviewer: mravishankar.
Herald added a reviewer: antiagainst.
Herald added projects: MLIR, LLVM.
rriddle requested review of this revision.
Herald added a reviewer: nicolasvasilache.
Herald added subscribers: stephenneuendorffer, nicolasvasilache.

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.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D91211

Files:
  llvm/include/llvm/ADT/Sequence.h
  mlir/include/mlir/Pass/AnalysisManager.h
  mlir/include/mlir/Pass/Pass.h
  mlir/include/mlir/Pass/PassManager.h
  mlir/include/mlir/Transforms/Passes.h
  mlir/include/mlir/Transforms/Passes.td
  mlir/lib/Pass/Pass.cpp
  mlir/lib/Pass/PassRegistry.cpp
  mlir/lib/Transforms/Inliner.cpp
  mlir/test/Dialect/Affine/inlining.mlir
  mlir/test/Dialect/SPIRV/Transforms/inlining.mlir
  mlir/test/Transforms/inlining.mlir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D91211.304333.patch
Type: text/x-patch
Size: 28974 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201110/549bcd2f/attachment.bin>