Add a new interface function to_string for ThinOrFullLTOPhase

Author: shiltian

llvm/include/llvm/Pass.h

@@ -86,6 +86,23 @@ enum class ThinOrFullLTOPhase {
   FullLTOPostLink
 };
 
+#ifndef NDEBUG
+static inline const char *to_string(ThinOrFullLTOPhase Phase) {
+  switch (Phase) {
+  case ThinOrFullLTOPhase::None:
+    return "None";
+  case ThinOrFullLTOPhase::ThinLTOPreLink:
+    return "ThinLTOPreLink";
+  case ThinOrFullLTOPhase::ThinLTOPostLink:
+    return "ThinLTOPostLink";
+  case ThinOrFullLTOPhase::FullLTOPreLink:
+    return "FullLTOPreLink";
+  case ThinOrFullLTOPhase::FullLTOPostLink:
+    return "FullLTOPostLink";
+  }
+}
+#endif
+
 //===----------------------------------------------------------------------===//
 /// Pass interface - Implemented by all 'passes'.  Subclass this if you are an
 /// interprocedural optimization or you do not fit into any of the more

llvm/lib/Target/AMDGPU/AMDGPUAttributor.cpp

@@ -1374,21 +1374,7 @@ static bool runImpl(Module &M, AnalysisGetter &AG, TargetMachine &TM,
   Attributor A(Functions, InfoCache, AC);
 
   LLVM_DEBUG({
-    auto PhaseToString = [](ThinOrFullLTOPhase LTOPhase) -> StringRef {
-      switch (LTOPhase) {
-      case ThinOrFullLTOPhase::None:
-        return "None";
-      case ThinOrFullLTOPhase::ThinLTOPreLink:
-        return "ThinLTOPreLink";
-      case ThinOrFullLTOPhase::ThinLTOPostLink:
-        return "ThinLTOPostLink";
-      case ThinOrFullLTOPhase::FullLTOPreLink:
-        return "FullLTOPreLink";
-      case ThinOrFullLTOPhase::FullLTOPostLink:
-        return "FullLTOPostLink";
-      }
-    };
-    StringRef LTOPhaseStr = PhaseToString(LTOPhase);
+    StringRef LTOPhaseStr = to_string(LTOPhase);
     dbgs() << "[AMDGPUAttributor] Running at phase " << LTOPhaseStr << '\n'
            << "[AMDGPUAttributor] Module " << M.getName() << " is "
            << (AC.IsClosedWorldModule ? "" : "not ")