[sil-printer] Refactor out the printing logic for printing of a SILLocation into its own method. NFC.

Author: gottesmm

lib/SIL/SILPrinter.cpp

@@ -549,99 +549,110 @@ class SILPrinter : public SILVisitor<SILPrinter> {
     return true;
   }
 
+  void printSILLocation(SILLocation L, SILModule &M, const SILDebugScope *DS,
+                        bool printedSlashes) {
+    if (!L.isNull()) {
+      if (!printedSlashes) {
+        PrintState.OS.PadToColumn(50);
+        *this << "//";
+      }
+      *this << " ";
+
+      // To minimize output, only print the line and column number for
+      // everything but the first instruction.
+      L.getSourceLoc().printLineAndColumn(PrintState.OS,
+                                          M.getASTContext().SourceMgr);
+
+      // Print the type of location.
+      switch (L.getKind()) {
+      case SILLocation::NoneKind:
+        assert(L.isAutoGenerated() && "This kind shouldn't be printed.");
+        break;
+      case SILLocation::RegularKind:
+        break;
+      case SILLocation::ReturnKind:
+        *this << ":return";
+        break;
+      case SILLocation::ImplicitReturnKind:
+        *this << ":imp_return";
+        break;
+      case SILLocation::InlinedKind:
+        *this << ":inlined";
+        break;
+      case SILLocation::MandatoryInlinedKind:
+        *this << ":minlined";
+        break;
+      case SILLocation::CleanupKind:
+        *this << ":cleanup";
+        break;
+      case SILLocation::ArtificialUnreachableKind:
+        *this << ":art_unreach";
+        break;
+      case SILLocation::SILFileKind:
+        *this << ":sil";
+        break;
+      }
+      if (L.isAutoGenerated())
+        *this << ":auto_gen";
+      if (L.isInPrologue())
+        *this << ":in_prologue";
+    }
+    if (L.isNull()) {
+      if (!printedSlashes) {
+        PrintState.OS.PadToColumn(50);
+        *this << "//";
+      }
+      if (L.isInTopLevel())
+        *this << " top_level";
+      else if (L.isAutoGenerated())
+        *this << " auto_gen";
+      else
+        *this << " no_loc";
+      if (L.isInPrologue())
+        *this << ":in_prologue";
+    }
+
+    // Print inlined-at location, if any.
+    if (DS) {
+      while (DS->InlinedCallSite) {
+        *this << ": perf_inlined_at ";
+        auto CallSite = DS->InlinedCallSite->Loc;
+        if (!CallSite.isNull())
+          CallSite.getSourceLoc().print(
+              PrintState.OS, M.getASTContext().SourceMgr, LastBufferID);
+        else
+          *this << "?";
+        DS = DS->InlinedCallSite;
+      }
+    }
+  }
+
   void print(SILValue V) {
     if (auto *FRI = dyn_cast<FunctionRefInst>(V))
       *this << "  // function_ref "
-         << demangleSymbolAsString(FRI->getReferencedFunction()->getName())
-         << "\n";
+            << demangleSymbolAsString(FRI->getReferencedFunction()->getName())
+            << "\n";
 
     *this << "  ";
 
     // Print result.
     if (V->hasValue()) {
       ID Name = getID(V);
-      Name.ResultNumber = -1;  // Don't print subresult number.
+      Name.ResultNumber = -1; // Don't print subresult number.
       *this << Name << " = ";
     }
 
     // Print the value.
     visit(V);
 
-    // Print users, or id for valueless instructions. Return true if we emitted
-    // any output.
+    // Print users, or id for valueless instructions.
     bool printedSlashes = printUsersOfSILValue(V);
 
     // Print SIL location.
     if (Verbose) {
-      if (SILInstruction *I = dyn_cast<SILInstruction>(V)) {
-        SILLocation L = I->getLoc();
-        SILModule &M = I->getModule();
-        if (!L.isNull()) {
-          if (!printedSlashes) {
-            PrintState.OS.PadToColumn(50);
-            *this << "//";
-          }
-          *this << " ";
-
-          // To minimize output, only print the line and column number for
-          // everything but the first instruction.
-          L.getSourceLoc().printLineAndColumn(PrintState.OS,
-                                              M.getASTContext().SourceMgr);
-
-          // Print the type of location.
-          switch (L.getKind()) {
-          case SILLocation::NoneKind :
-            assert(L.isAutoGenerated() && "This kind shouldn't be printed.");
-            break;
-          case SILLocation::RegularKind :
-            break;
-          case SILLocation::ReturnKind :
-            *this << ":return"; break;
-          case SILLocation::ImplicitReturnKind :
-            *this << ":imp_return"; break;
-          case SILLocation::InlinedKind :
-            *this << ":inlined"; break;
-          case SILLocation::MandatoryInlinedKind :
-            *this << ":minlined"; break;
-          case SILLocation::CleanupKind :
-            *this << ":cleanup"; break;
-          case SILLocation::ArtificialUnreachableKind :
-            *this << ":art_unreach"; break;
-          case SILLocation::SILFileKind :
-            *this << ":sil"; break;
-          }
-          if (L.isAutoGenerated())
-            *this << ":auto_gen";
-          if (L.isInPrologue())
-            *this << ":in_prologue";
-        }
-        if (L.isNull()) {
-          if (!printedSlashes) {
-            PrintState.OS.PadToColumn(50);
-            *this << "//";
-          }
-          if (L.isInTopLevel())
-            *this << " top_level";
-          else if (L.isAutoGenerated())
-            *this << " auto_gen";
-          else
-            *this << " no_loc";
-          if (L.isInPrologue())
-            *this << ":in_prologue";
-        }
-
-        // Print inlined-at location, if any.
-        if (auto DS = I->getDebugScope())
-          while (DS->InlinedCallSite) {
-            *this << ": perf_inlined_at ";
-            auto CallSite = DS->InlinedCallSite->Loc;
-            if (!CallSite.isNull())
-              CallSite.getSourceLoc().
-                print(PrintState.OS, M.getASTContext().SourceMgr, LastBufferID);
-            else
-              *this << "?";
-            DS = DS->InlinedCallSite;
-          }
+      if (auto *I = dyn_cast<SILInstruction>(V)) {
+        printSILLocation(I->getLoc(), I->getModule(), I->getDebugScope(),
+                         printedSlashes);
       }
     }