[flang-commits] [flang] [llvm] [mlir] [flang][OpenMP] Enable tiling (PR #143715)

[Michael Kruse via flang-commits]

================
@@ -10845,3 +10862,10 @@ void CanonicalLoopInfo::invalidate() {
   Latch = nullptr;
   Exit = nullptr;
 }
+
+void CanonicalLoopInfo::dump() {
+  errs() << "CanonicaLoop == Header: " << (Header ? Header->getName() : "n/a")
+         << " Cond: " << (Cond ? Cond->getName() : "n/a")
+         << " Latch: " << (Latch ? Latch->getName() : "n/a")
+         << " Exit: " << (Exit ? Exit->getName() : "n/a") << "\n";
+}
----------------
Meinersbur wrote:

```suggestion
#if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP)
LLVM_DUMP_METHOD void CanonicalLoopInfo::dump() {
  errs() << "CanonicaLoop == Header: " << (Header ? Header->getName() : "n/a")
         << " Cond: " << (Cond ? Cond->getName() : "n/a")
         << " Latch: " << (Latch ? Latch->getName() : "n/a")
         << " Exit: " << (Exit ? Exit->getName() : "n/a") << "\n";
}
#endif
```
There is a pattern for dump functions. For avoiding it being removed despite unused in optimized builds an adding it in non-assert builds.

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