[all-commits] [llvm/llvm-project] 94a309: [mlir][Pass] Make PassManager default to op-agnostic

[rk via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 94a309284d722f2cb58c91d878c891cf54039f2e
      https://github.com/llvm/llvm-project/commit/94a309284d722f2cb58c91d878c891cf54039f2e
  Author: rkayaith <rkayaith at gmail.com>
  Date:   2023-01-25 (Wed, 25 Jan 2023)

  Changed paths:
    M flang/lib/Frontend/FrontendActions.cpp
    M flang/tools/bbc/bbc.cpp
    M flang/tools/tco/tco.cpp
    M mlir/docs/PassManagement.md
    M mlir/docs/Tutorials/Toy/Ch-3.md
    M mlir/examples/toy/Ch3/toyc.cpp
    M mlir/examples/toy/Ch4/toyc.cpp
    M mlir/examples/toy/Ch5/toyc.cpp
    M mlir/examples/toy/Ch6/toyc.cpp
    M mlir/examples/toy/Ch7/toyc.cpp
    M mlir/include/mlir/IR/OperationSupport.h
    M mlir/include/mlir/Pass/PassManager.h
    M mlir/lib/Pass/Pass.cpp
    M mlir/lib/Rewrite/FrozenRewritePatternSet.cpp
    M mlir/lib/Tools/mlir-opt/MlirOptMain.cpp
    M mlir/unittests/ExecutionEngine/Invoke.cpp
    M mlir/unittests/Pass/PassManagerTest.cpp

  Log Message:
  -----------
  [mlir][Pass] Make PassManager default to op-agnostic

Currently `PassManager` defaults to being anchored on `builtin.module`.
Switching the default makes `PassManager` consistent with
`OpPassManager` and avoids the implicit dependency on `builtin.module`.

Specifying the anchor op type isn't strictly necessary when using
explicit nesting (existing pipelines will continue to work), but I've
updated most call sites to specify the anchor since it allows for better
error-checking during pipeline construction.

Reviewed By: rriddle

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