commit d0644fa28031eaa54bff4819fb165830ea87c719
author Matt Arsenault <Matthew.Arsenault@amd.com> Mon Jun 02 16:03:38 2025 +0200
committer Matt Arsenault <arsenm2@gmail.com> Tue Jun 03 14:55:02 2025 +0900
tree 568f286d4c69f94a5ee57893503d1f864326919d
parent 52efdd8cb4cda57363a5bf868cae7ca82bd04513

llvm-tli-checker: Avoid a temporary string while printing

Directly write to the output instead of building a string to
print.

llvm/tools/llvm-tli-checker/llvm-tli-checker.cpp

diff --git a/llvm/tools/llvm-tli-checker/llvm-tli-checker.cpp b/llvm/tools/llvm-tli-checker/llvm-tli-checker.cpp
index 725fe71..5d58115 100644
--- a/llvm/tools/llvm-tli-checker/llvm-tli-checker.cpp
+++ b/llvm/tools/llvm-tli-checker/llvm-tli-checker.cpp
@@ -98,16 +98,12 @@
 }
 
 // Return Name, and if Name is mangled, append "aka" and the demangled name.
-static std::string getPrintableName(StringRef Name) {
-  std::string OutputName = "'";
-  OutputName += Name;
-  OutputName += "'";
+static void printPrintableName(raw_ostream &OS, StringRef Name) {
+  OS << '\'' << Name << '\'';
+
   std::string DemangledName(demangle(Name));
-  if (Name != DemangledName) {
-    OutputName += " aka ";
-    OutputName += DemangledName;
-  }
-  return OutputName;
+  if (Name != DemangledName)
+    OS << " aka " << DemangledName;
 }
 
 static void reportNumberOfEntries(const TargetLibraryInfo &TLI,
@@ -138,10 +134,10 @@
       StringRef Name = TLI.getName(LF);
       // If there is a custom name, print it.
       // TODO: Should we include the standard name in the printed line?
-      outs() << getPrintableName(Name);
+      printPrintableName(outs(), Name);
     } else {
       // If it's not available, refer to it by the standard name.
-      outs() << getPrintableName(TargetLibraryInfo::getStandardName(LF));
+      printPrintableName(outs(), TargetLibraryInfo::getStandardName(LF));
     }
 
     outs() << '\n';
@@ -345,7 +341,9 @@
         constexpr char YesNo[2][4] = {"no ", "yes"};
         constexpr char Indicator[4][3] = {"!!", ">>", "<<", "=="};
         outs() << Indicator[Which] << " TLI " << YesNo[TLIHas] << " SDK "
-               << YesNo[SDKHas] << ": " << getPrintableName(TLIName) << '\n';
+               << YesNo[SDKHas] << ": ";
+        printPrintableName(outs(), TLIName);
+        outs() << '\n';
       }
     }