[Mlir-commits] [mlir] [SCF] add debug label for MergeNestedParallelLoops pattern (PR #135980)

[Mehdi Amini]

joker-eph wrote:

The problem I see here is that you want to attach semantics through the debug name, however here is how they are documented:

```
  /// Return a readable name for this pattern. This name should only be used for
  /// debugging purposes, and may be empty.
  StringRef getDebugName() const { return debugName; }

  /// Set the human readable debug name used for this pattern. This name will
  /// only be used for debugging purposes.
  void setDebugName(StringRef name) { debugName = name; }
```

That means we should be able to strip these out of a release build for example without breaking anything, which you may object to in the future if you depends on this behavior.

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