Delete ActionManager and replace it with a simple callback on the Context
authorMehdi Amini <joker.eph@gmail.com>
Sat, 4 Feb 2023 22:16:35 +0000 (14:16 -0800)
committerMehdi Amini <joker.eph@gmail.com>
Tue, 7 Mar 2023 07:25:34 +0000 (08:25 +0100)
commit9b1fe5649e546863adba3fdde158b50a213ea954
treecfdb34dbf08b5b8c41d4e324cede9a64bcfd5f2f
parent89a515d7b1e87189cffee02b7edbf5f4fd989fd9
Delete ActionManager and replace it with a simple callback on the Context

The concept of the ActionManager acts as a sort of "Hub" that can receive
various types of action and dispatch them to a set of registered handlers.
One handler will handle the action or it'll cascade to other handlers.

This model does not really fit the current evolution of the Action tracing
and debugging: we can't foresee a good case where this behavior compose with
the use-case behind the handlers. Instead we simplify it with a single
callback installed on the Context.

Differential Revision: https://reviews.llvm.org/D144811
mlir/include/mlir/Debug/Counter.h
mlir/include/mlir/IR/Action.h
mlir/include/mlir/IR/MLIRContext.h
mlir/lib/Debug/DebugCounter.cpp
mlir/lib/IR/MLIRContext.cpp
mlir/lib/Tools/mlir-opt/MlirOptMain.cpp
mlir/unittests/Debug/DebugCounterTest.cpp
mlir/unittests/IR/ActionTest.cpp [deleted file]
mlir/unittests/IR/CMakeLists.txt