Skip to content

Commit d60b2b2

Browse files
authored
[llvm][DebugInfo] Use formatv in GsymReaderV1 and GsymReaderV2 (#192000)
This relates to #35980.
1 parent f943685 commit d60b2b2

2 files changed

Lines changed: 9 additions & 7 deletions

File tree

llvm/lib/DebugInfo/GSYM/GsymReaderV1.cpp

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313

1414
#include "llvm/DebugInfo/GSYM/GsymDataExtractor.h"
1515
#include "llvm/DebugInfo/GSYM/Header.h"
16+
#include "llvm/Support/FormatVariadic.h"
1617
#include "llvm/Support/MemoryBuffer.h"
1718

1819
using namespace llvm;
@@ -94,7 +95,7 @@ void GsymReaderV1::dump(raw_ostream &OS) {
9495
OS << " (ADDRESS)\n";
9596
OS << "====== =============================== \n";
9697
for (uint32_t I = 0; I < getNumAddresses(); ++I) {
97-
OS << format("[%4u] ", I);
98+
OS << formatv("[{0,4}] ", I);
9899
switch (getAddressOffsetSize()) {
99100
case 1:
100101
OS << HEX8(getAddrOffsets<uint8_t>()[I]);
@@ -117,15 +118,15 @@ void GsymReaderV1::dump(raw_ostream &OS) {
117118
OS << "INDEX Offset\n";
118119
OS << "====== ==========\n";
119120
for (uint32_t I = 0; I < getNumAddresses(); ++I)
120-
OS << format("[%4u] ", I) << HEX32(*getAddressInfoOffset(I)) << "\n";
121+
OS << formatv("[{0,4}] ", I) << HEX32(*getAddressInfoOffset(I)) << "\n";
121122
OS << "\nFiles:\n";
122123
OS << "INDEX DIRECTORY BASENAME PATH\n";
123124
OS << "====== ========== ========== ==============================\n";
124125
for (uint32_t I = 0;; ++I) {
125126
auto FE = getFile(I);
126127
if (!FE)
127128
break;
128-
OS << format("[%4u] ", I) << HEX32(FE->Dir) << ' ' << HEX32(FE->Base)
129+
OS << formatv("[{0,4}] ", I) << HEX32(FE->Dir) << ' ' << HEX32(FE->Base)
129130
<< ' ';
130131
dump(OS, FE);
131132
OS << "\n";

llvm/lib/DebugInfo/GSYM/GsymReaderV2.cpp

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@
1414
#include "llvm/ADT/STLExtras.h"
1515
#include "llvm/DebugInfo/GSYM/GlobalData.h"
1616
#include "llvm/DebugInfo/GSYM/GsymDataExtractor.h"
17+
#include "llvm/Support/FormatVariadic.h"
1718
#include "llvm/Support/MemoryBuffer.h"
1819

1920
using namespace llvm;
@@ -48,7 +49,7 @@ void GsymReaderV2::dump(raw_ostream &OS) {
4849
assert(GDOrErr && "GlobalData::decode() should not fail");
4950
const GlobalData &GD = *GDOrErr;
5051

51-
OS << format("%-15s ", getNameForGlobalInfoType(GD.Type).data())
52+
OS << formatv("{0,-15} ", getNameForGlobalInfoType(GD.Type).data())
5253
<< HEX64(GD.FileOffset) << " " << HEX64(GD.FileSize) << "\n";
5354

5455
// Stop printing after the end of list entry.
@@ -87,7 +88,7 @@ void GsymReaderV2::dump(raw_ostream &OS) {
8788
OS << " (ADDRESS 64)\n";
8889
OS << "====== ========================================\n";
8990
for (uint32_t I = 0; I < getNumAddresses(); ++I) {
90-
OS << format("[%4u] ", I);
91+
OS << formatv("[{0,4}] ", I);
9192
switch (getAddressOffsetSize()) {
9293
case 1:
9394
OS << HEX8(getAddrOffsets<uint8_t>()[I]);
@@ -113,7 +114,7 @@ void GsymReaderV2::dump(raw_ostream &OS) {
113114
uint64_t RelOffset = I * getAddressInfoOffsetSize();
114115
uint64_t RelValue =
115116
AddrInfoOffsetsData.getUnsigned(&RelOffset, getAddressInfoOffsetSize());
116-
OS << format("[%4u] ", I) << HEX64(RelValue) << " ("
117+
OS << formatv("[{0,4}] ", I) << HEX64(RelValue) << " ("
117118
<< HEX64(*getAddressInfoOffset(I)) << ")\n";
118119
}
119120
OS << "\nFiles:\n";
@@ -126,7 +127,7 @@ void GsymReaderV2::dump(raw_ostream &OS) {
126127
auto FE = getFile(I);
127128
if (!FE)
128129
break;
129-
OS << format("[%4u] ", I) << HEX32(FE->Dir) << ' ' << HEX32(FE->Base)
130+
OS << formatv("[{0,4}] ", I) << HEX32(FE->Dir) << ' ' << HEX32(FE->Base)
130131
<< ' ';
131132
dump(OS, FE);
132133
OS << "\n";

0 commit comments

Comments
 (0)