[llvm] [llvm][DebugInfo] Use formatv in GsymReader (PR #192000)

Konrad Kleine via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 16 00:33:50 PDT 2026


https://github.com/kwk updated https://github.com/llvm/llvm-project/pull/192000

>From 2c16d06eb741469ba021f705a727b564cc45201b Mon Sep 17 00:00:00 2001
From: Konrad Kleine <kkleine at redhat.com>
Date: Thu, 16 Apr 2026 07:30:59 +0000
Subject: [PATCH] [llvm][DebugInfo] Use formatv in GsymReader

This relates to #35980.
---
 llvm/lib/DebugInfo/GSYM/GsymReaderV1.cpp | 7 ++++---
 llvm/lib/DebugInfo/GSYM/GsymReaderV2.cpp | 9 +++++----
 2 files changed, 9 insertions(+), 7 deletions(-)

diff --git a/llvm/lib/DebugInfo/GSYM/GsymReaderV1.cpp b/llvm/lib/DebugInfo/GSYM/GsymReaderV1.cpp
index 51483e25882e7..68150fb179e67 100644
--- a/llvm/lib/DebugInfo/GSYM/GsymReaderV1.cpp
+++ b/llvm/lib/DebugInfo/GSYM/GsymReaderV1.cpp
@@ -13,6 +13,7 @@
 
 #include "llvm/DebugInfo/GSYM/GsymDataExtractor.h"
 #include "llvm/DebugInfo/GSYM/Header.h"
+#include "llvm/Support/FormatVariadic.h"
 #include "llvm/Support/MemoryBuffer.h"
 
 using namespace llvm;
@@ -94,7 +95,7 @@ void GsymReaderV1::dump(raw_ostream &OS) {
   OS << " (ADDRESS)\n";
   OS << "====== =============================== \n";
   for (uint32_t I = 0; I < getNumAddresses(); ++I) {
-    OS << format("[%4u] ", I);
+    OS << formatv("[{0,4}] ", I);
     switch (getAddressOffsetSize()) {
     case 1:
       OS << HEX8(getAddrOffsets<uint8_t>()[I]);
@@ -117,7 +118,7 @@ void GsymReaderV1::dump(raw_ostream &OS) {
   OS << "INDEX  Offset\n";
   OS << "====== ==========\n";
   for (uint32_t I = 0; I < getNumAddresses(); ++I)
-    OS << format("[%4u] ", I) << HEX32(*getAddressInfoOffset(I)) << "\n";
+    OS << formatv("[{0,4}] ", I) << HEX32(*getAddressInfoOffset(I)) << "\n";
   OS << "\nFiles:\n";
   OS << "INDEX  DIRECTORY  BASENAME   PATH\n";
   OS << "====== ========== ========== ==============================\n";
@@ -125,7 +126,7 @@ void GsymReaderV1::dump(raw_ostream &OS) {
     auto FE = getFile(I);
     if (!FE)
       break;
-    OS << format("[%4u] ", I) << HEX32(FE->Dir) << ' ' << HEX32(FE->Base)
+    OS << formatv("[{0,4}] ", I) << HEX32(FE->Dir) << ' ' << HEX32(FE->Base)
        << ' ';
     dump(OS, FE);
     OS << "\n";
diff --git a/llvm/lib/DebugInfo/GSYM/GsymReaderV2.cpp b/llvm/lib/DebugInfo/GSYM/GsymReaderV2.cpp
index 376dbb17d26d7..1a8dfcb5f4038 100644
--- a/llvm/lib/DebugInfo/GSYM/GsymReaderV2.cpp
+++ b/llvm/lib/DebugInfo/GSYM/GsymReaderV2.cpp
@@ -14,6 +14,7 @@
 #include "llvm/ADT/STLExtras.h"
 #include "llvm/DebugInfo/GSYM/GlobalData.h"
 #include "llvm/DebugInfo/GSYM/GsymDataExtractor.h"
+#include "llvm/Support/FormatVariadic.h"
 #include "llvm/Support/MemoryBuffer.h"
 
 using namespace llvm;
@@ -48,7 +49,7 @@ void GsymReaderV2::dump(raw_ostream &OS) {
     assert(GDOrErr && "GlobalData::decode() should not fail");
     const GlobalData &GD = *GDOrErr;
 
-    OS << format("%-15s ", getNameForGlobalInfoType(GD.Type).data())
+    OS << formatv("{0,-15} ", getNameForGlobalInfoType(GD.Type).data())
        << HEX64(GD.FileOffset) << "  " << HEX64(GD.FileSize) << "\n";
 
     // Stop printing after the end of list entry.
@@ -87,7 +88,7 @@ void GsymReaderV2::dump(raw_ostream &OS) {
   OS << " (ADDRESS 64)\n";
   OS << "====== ========================================\n";
   for (uint32_t I = 0; I < getNumAddresses(); ++I) {
-    OS << format("[%4u] ", I);
+    OS << formatv("[{0,4}] ", I);
     switch (getAddressOffsetSize()) {
     case 1:
       OS << HEX8(getAddrOffsets<uint8_t>()[I]);
@@ -113,7 +114,7 @@ void GsymReaderV2::dump(raw_ostream &OS) {
     uint64_t RelOffset = I * getAddressInfoOffsetSize();
     uint64_t RelValue =
         AddrInfoOffsetsData.getUnsigned(&RelOffset, getAddressInfoOffsetSize());
-    OS << format("[%4u] ", I) << HEX64(RelValue) << " ("
+    OS << formatv("[{0,4}] ", I) << HEX64(RelValue) << " ("
        << HEX64(*getAddressInfoOffset(I)) << ")\n";
   }
   OS << "\nFiles:\n";
@@ -126,7 +127,7 @@ void GsymReaderV2::dump(raw_ostream &OS) {
     auto FE = getFile(I);
     if (!FE)
       break;
-    OS << format("[%4u] ", I) << HEX32(FE->Dir) << ' ' << HEX32(FE->Base)
+    OS << formatv("[{0,4}] ", I) << HEX32(FE->Dir) << ' ' << HEX32(FE->Base)
        << ' ';
     dump(OS, FE);
     OS << "\n";



More information about the llvm-commits mailing list