[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