[Mlir-commits] [mlir] [mlir] Switch to new LDBG macro (PR #150616)

[Han-Chung Wang]

hanhanW wrote:

I think this breaks Release build, because `DEBUGLOG_WITH_STREAM_AND_TYPE` is not defined in release build, and the transform interface still use it in Release build?

https://github.com/llvm/llvm-project/blob/80594820512605f25993b28d0aab2214d5308d59/llvm/include/llvm/Support/DebugLog.h#L19-L32

https://github.com/llvm/llvm-project/blob/80594820512605f25993b28d0aab2214d5308d59/mlir/lib/Dialect/Transform/Interfaces/TransformInterfaces.cpp#L23-L28

I'm seeing the below error on my local Release build, but not Debug build.

```
llvm-project/mlir/lib/Dialect/Transform/Interfaces/TransformInterfaces.cpp:27:22: error: ‘DEBUGLOG_WITH_STREAM_AND_TYPE’ was not declared in this scope
   27 | #define FULL_LDBG(X) DEBUGLOG_WITH_STREAM_AND_TYPE(llvm::dbgs(), DEBUG_TYPE_FULL)                                                                                                                                                                          |                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
llvm-project/mlir/lib/Dialect/Transform/Interfaces/TransformInterfaces.cpp:489:3: note: in expansion of macro ‘FULL_LDBG’
  489 |   FULL_LDBG() << "--recordOpHandleInvalidationOne";
```

https://github.com/llvm/llvm-project/pull/150616