Skip to content

Commit 54b2004

Browse files
committed
Address review comments.
1 parent 72b0d37 commit 54b2004

File tree

4 files changed

+15
-14
lines changed

4 files changed

+15
-14
lines changed

llvm/test/tools/llvm-readobj/ELF/call-graph-info-err-missing-indirect.test

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
## Tests that --call-graph-info fails if .callgraph section does not have
1+
## Tests that --call-graph-info fails if .llvm.callgraph section does not have
22
## an expected value, e.g., not as much call sites as the given count.
33

44
# REQUIRES: x86-registered-target

llvm/test/tools/llvm-readobj/ELF/call-graph-info-err-no-callgraph-section.test

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,11 +3,11 @@
33
# REQUIRES: x86-registered-target
44

55
# RUN: llvm-mc %s -filetype=obj -triple=x86_64-pc-linux -o %t
6-
# RUN: not llvm-readelf --call-graph-info %t 2>&1 | FileCheck %s --match-full-lines -DFILE=%t
7-
# RUN: not llvm-readelf --elf-output-style=LLVM --call-graph-info %t 2>&1 | FileCheck %s --match-full-lines -DFILE=%t
8-
# RUN: not llvm-readelf --elf-output-style=JSON --pretty-print --call-graph-info %t 2>&1 | FileCheck %s --match-full-lines -DFILE=%t
6+
# RUN: not llvm-readelf --call-graph-info %t 2>&1 | FileCheck %s -DFILE=%t
7+
# RUN: not llvm-readelf --elf-output-style=LLVM --call-graph-info %t 2>&1 | FileCheck %s -DFILE=%t
8+
# RUN: not llvm-readelf --elf-output-style=JSON --pretty-print --call-graph-info %t 2>&1 | FileCheck %s -DFILE=%t
99

10-
# CHECK: {{.*}}llvm-readelf{{.*}}: error: 'Missing section': No .llvm.callgraph section found.
10+
# CHECK: error: 'Missing section': No .llvm.callgraph section found.
1111

1212
.text
1313
.globl _Z3foov

llvm/test/tools/llvm-readobj/ELF/call-graph-info.test

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -44,34 +44,34 @@
4444
# CHECK: Call graph section '.llvm.callgraph' contains 4 entries:
4545
# CHECK-EMPTY:
4646
# CHECK-NEXT: Entry 0:
47-
# CHECK-NEXT: Address: 0x1790 <foo>
47+
# CHECK-NEXT: Address: 0x1790 <foo>
4848
# CHECK-NEXT: Indirect Target: Yes
4949
# CHECK-NEXT: Type ID: 0x3ecbeef531f74424
5050
# CHECK-NEXT: Direct Callees (0):
5151
# CHECK-NEXT: Indirect Callees by Type ID (0):
5252
# CHECK-EMPTY:
5353
# CHECK-NEXT: Entry 1:
54-
# CHECK-NEXT: Address: 0x17a0 <bar>
54+
# CHECK-NEXT: Address: 0x17a0 <bar>
5555
# CHECK-NEXT: Indirect Target: Yes
5656
# CHECK-NEXT: Type ID: 0x3ecbeef531f74424
5757
# CHECK-NEXT: Direct Callees (0):
5858
# CHECK-NEXT: Indirect Callees by Type ID (0):
5959
# CHECK-EMPTY:
6060
# CHECK-NEXT: Entry 2:
61-
# CHECK-NEXT: Address: 0x17b0 <baz>
61+
# CHECK-NEXT: Address: 0x17b0 <baz>
6262
# CHECK-NEXT: Indirect Target: Yes
6363
# CHECK-NEXT: Type ID: 0x308e4b8159bc8654
6464
# CHECK-NEXT: Direct Callees (0):
6565
# CHECK-NEXT: Indirect Callees by Type ID (0):
6666
# CHECK-EMPTY:
6767
# CHECK-NEXT: Entry 3:
68-
# CHECK-NEXT: Address: 0x17c0 <main>
68+
# CHECK-NEXT: Address: 0x17c0 <main>
6969
# CHECK-NEXT: Indirect Target: Yes
7070
# CHECK-NEXT: Type ID: 0xfa6809609a76afca
7171
# CHECK-NEXT: Direct Callees (3):
72-
# CHECK-NEXT: Address: 0x1790 <foo>
73-
# CHECK-NEXT: Address: 0x17a0 <bar>
74-
# CHECK-NEXT: Address: 0x17b0 <baz>
72+
# CHECK-NEXT: Address: 0x1790 <foo>
73+
# CHECK-NEXT: Address: 0x17a0 <bar>
74+
# CHECK-NEXT: Address: 0x17b0 <baz>
7575
# CHECK-NEXT: Indirect Callees by Type ID (2):
7676
# CHECK-NEXT: 0x3ecbeef531f74424
7777
# CHECK-NEXT: 0x308e4b8159bc8654
@@ -584,7 +584,7 @@ Sections:
584584
EntSize: 0x1
585585
Content: 004675636873696120636C616E672076657273696F6E2032322E302E306769742028676974406769746875622E636F6D3A5072616268756B2F6C6C766D2D70726F6A6563742E676974203934396232653930373439623765376438353739663838626231303366373535353832393465666229004C696E6B65723A2046756368736961204C4C442032322E302E302028676974406769746875622E636F6D3A5072616268756B2F6C6C766D2D70726F6A6563742E676974203934396232653930373439623765376438353739663838626231303366373535353832393465666229004675636873696120636C616E672076657273696F6E2032322E302E306769742028676974406769746875622E636F6D3A5072616268756B2F6C6C766D2D70726F6A6563742E676974203861333966396231386139343632386138656463666433613137363432376364333161656330633829004675636873696120636C616E672076657273696F6E2032322E302E306769742028676974406769746875622E636F6D3A5072616268756B2F6C6C766D2D70726F6A6563742E67697420653038356135633833363436316632396437336637333933343334343765643964353338626563312900
586586
- Name: .llvm.callgraph
587-
Type: 0x6FFF4C0F
587+
Type: SHT_LLVM_CALL_GRAPH
588588
Flags: [ SHF_LINK_ORDER ]
589589
Link: .text
590590
AddressAlign: 0x1

llvm/tools/llvm-readobj/ELFDumper.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5497,7 +5497,8 @@ template <class ELFT> void GNUELFDumper<ELFT>::printCallGraphInfo() {
54975497
auto PrintFunc = [](uint64_t FuncEntryPC, std::string FuncSymName,
54985498
formatted_raw_ostream &OS) {
54995499
OS.PadToColumn(4);
5500-
OS << "Address: ";
5500+
OS << "Address:";
5501+
OS.PadToColumn(21);
55015502
OS << to_string(format_hex(FuncEntryPC, 1));
55025503
if (!FuncSymName.empty())
55035504
OS << " <" << FuncSymName << ">";

0 commit comments

Comments
 (0)