[all-commits] [llvm/llvm-project] ae3fae: [MLIR][mlir-opt] move action debugger hook flag (#...

[Christopher McGirr via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: ae3faea1f28f840bddd819d1c45e7f7d3e75703c
      https://github.com/llvm/llvm-project/commit/ae3faea1f28f840bddd819d1c45e7f7d3e75703c
  Author: Christopher McGirr <7071833+chrsmcgrr at users.noreply.github.com>
  Date:   2025-04-08 (Tue, 08 Apr 2025)

  Changed paths:
    M mlir/include/mlir/Tools/mlir-opt/MlirOptMain.h
    M mlir/lib/Debug/CLOptionsSetup.cpp
    M mlir/lib/Tools/mlir-opt/MlirOptMain.cpp
    A mlir/test/mlir-opt/debuggerhook.mlir

  Log Message:
  -----------
  [MLIR][mlir-opt] move action debugger hook flag (#134842)

Currently if a developer uses the flag `--mlir-enable-debugger-hook` the
debugger hook is not actually enabled. It seems the DebugConfig and the
MainMLIROptConfig are not connected.

To fix this we can move the `enableDebuggerHook` CL Option to the
DebugConfigCLOptions struct so that it can get registered and enabled
along with the other debugger flags. AFAICS there are no other uses of
the flag so this should be safe.

This also adds a small LIT test to check that the hook is enabled by
checking the std::cerr output for the log message.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications