[mlir] Split things dependent on LLVM_DEBUG into a .cpp file

LLVM_DEBUG in headers is awkward, better avoid it. DEBUG_TYPE in a
header results in a lot of macro redefinition warnings.

Author: d0k

mlir/include/mlir/Support/InterfaceSupport.h

@@ -15,11 +15,8 @@
 
 #include "mlir/Support/TypeID.h"
 #include "llvm/ADT/DenseMap.h"
-#include "llvm/Support/Debug.h"
 #include "llvm/Support/TypeName.h"
 
-#define DEBUG_TYPE "interfaces"
-
 namespace mlir {
 namespace detail {
 //===----------------------------------------------------------------------===//
@@ -231,19 +228,7 @@ class InterfaceMap {
     std::pair<TypeID, void *> elements[] = {
         std::make_pair(IfaceModels::Interface::getInterfaceID(),
                        new (malloc(sizeof(IfaceModels))) IfaceModels())...};
-    // Insert directly into the right position to keep the interfaces sorted.
-    for (auto &element : elements) {
-      TypeID id = element.first;
-      auto it =
-          llvm::lower_bound(interfaces, id, [](const auto &it, TypeID id) {
-            return compare(it.first, id);
-          });
-      if (it != interfaces.end() && it->first == id) {
-        LLVM_DEBUG(llvm::dbgs() << "Ignoring repeated interface registration");
-        continue;
-      }
-      interfaces.insert(it, element);
-    }
+    insert(elements);
   }
 
 private:
@@ -254,6 +239,8 @@ class InterfaceMap {
 
   InterfaceMap() = default;
 
+  void insert(ArrayRef<std::pair<TypeID, void *>> elements);
+
   template <typename... Ts>
   static InterfaceMap getImpl(std::tuple<Ts...> *) {
     std::pair<TypeID, void *> elements[] = {std::make_pair(

mlir/lib/Support/CMakeLists.txt

@@ -2,6 +2,7 @@ set(LLVM_OPTIONAL_SOURCES
   DebugCounter.cpp
   FileUtilities.cpp
   IndentedOstream.cpp
+  InterfaceSupport.cpp
   MlirOptMain.cpp
   StorageUniquer.cpp
   Timing.cpp
@@ -11,6 +12,7 @@ set(LLVM_OPTIONAL_SOURCES
 add_mlir_library(MLIRSupport
   DebugCounter.cpp
   FileUtilities.cpp
+  InterfaceSupport.cpp
   StorageUniquer.cpp
   Timing.cpp
   ToolUtilities.cpp

mlir/lib/Support/InterfaceSupport.cpp

@@ -0,0 +1,35 @@
+//===- InterfaceSupport.cpp - MLIR Interface Support Classes --------------===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+//
+//===----------------------------------------------------------------------===//
+//
+// This file defines several support classes for defining interfaces.
+//
+//===----------------------------------------------------------------------===//
+
+#include "mlir/Support/InterfaceSupport.h"
+#include "llvm/Support/Debug.h"
+#include "llvm/Support/raw_ostream.h"
+
+#define DEBUG_TYPE "interfaces"
+
+using namespace mlir;
+
+void detail::InterfaceMap::insert(
+    ArrayRef<std::pair<TypeID, void *>> elements) {
+  // Insert directly into the right position to keep the interfaces sorted.
+  for (auto &element : elements) {
+    TypeID id = element.first;
+    auto *it = llvm::lower_bound(interfaces, id, [](const auto &it, TypeID id) {
+      return compare(it.first, id);
+    });
+    if (it != interfaces.end() && it->first == id) {
+      LLVM_DEBUG(llvm::dbgs() << "Ignoring repeated interface registration");
+      continue;
+    }
+    interfaces.insert(it, element);
+  }
+}