From 5f7ade26d01f779fcec92d428ca738f8d12bc86a Mon Sep 17 00:00:00 2001 From: Rafael Espindola Date: Tue, 23 Jun 2015 15:45:38 +0000 Subject: [PATCH] objdump: Don't print a (always 0) size for MachO symbols. Only common symbol on MachO and COFF have a size. For COFF we already had a custom format. For MachO, there is no native objdump and we were printing it as ELF. Now we only print the sizes for symbols that actually have them. llvm-svn: 240422 --- llvm/test/Object/objdump-symbol-table.test | 6 +++--- .../tools/llvm-objdump/X86/macho-symbol-table.test | 10 +++++----- llvm/tools/llvm-objdump/llvm-objdump.cpp | 20 ++++++++++---------- 3 files changed, 18 insertions(+), 18 deletions(-) diff --git a/llvm/test/Object/objdump-symbol-table.test b/llvm/test/Object/objdump-symbol-table.test index 3d09e1a..e66faec 100644 --- a/llvm/test/Object/objdump-symbol-table.test +++ b/llvm/test/Object/objdump-symbol-table.test @@ -30,9 +30,9 @@ ELF-i386: 00000000 *UND* 00000000 puts macho-i386: trivial-object-test.macho-i386: file format Mach-O 32-bit i386 macho-i386: SYMBOL TABLE: -macho-i386: 00000000 g F __TEXT,__text 00000000 _main -macho-i386: 00000000 *UND* 00000000 _SomeOtherFunction -macho-i386: 00000000 *UND* 00000000 _puts +macho-i386: 00000000 g F __TEXT,__text _main +macho-i386: 00000000 *UND* _SomeOtherFunction +macho-i386: 00000000 *UND* _puts ELF-shared: shared-object-test.elf-i386: file format ELF-shared: SYMBOL TABLE: diff --git a/llvm/test/tools/llvm-objdump/X86/macho-symbol-table.test b/llvm/test/tools/llvm-objdump/X86/macho-symbol-table.test index 3fe5aea..826d78a 100644 --- a/llvm/test/tools/llvm-objdump/X86/macho-symbol-table.test +++ b/llvm/test/tools/llvm-objdump/X86/macho-symbol-table.test @@ -1,8 +1,8 @@ RUN: llvm-objdump -macho -t %p/Inputs/hello.obj.macho-x86_64 | FileCheck %s CHECK: SYMBOL TABLE: -CHECK: 000000000000003b l F __TEXT,__cstring 00000000 L_.str -CHECK: 0000000000000068 l F __TEXT,__eh_frame 00000000 EH_frame0 -CHECK: 0000000000000000 g F __TEXT,__text 00000000 _main -CHECK: 0000000000000080 g F __TEXT,__eh_frame 00000000 _main.eh -CHECK: 0000000000000000 *UND* 00000000 _printf +CHECK: 000000000000003b l F __TEXT,__cstring L_.str +CHECK: 0000000000000068 l F __TEXT,__eh_frame EH_frame0 +CHECK: 0000000000000000 g F __TEXT,__text _main +CHECK: 0000000000000080 g F __TEXT,__eh_frame _main.eh +CHECK: 0000000000000000 *UND* _printf diff --git a/llvm/tools/llvm-objdump/llvm-objdump.cpp b/llvm/tools/llvm-objdump/llvm-objdump.cpp index dd484f2..77e95f6 100644 --- a/llvm/tools/llvm-objdump/llvm-objdump.cpp +++ b/llvm/tools/llvm-objdump/llvm-objdump.cpp @@ -1085,7 +1085,6 @@ void llvm::PrintSymbolTable(const ObjectFile *o) { continue; if (error(Symbol.getType(Type))) continue; - uint64_t Size = Symbol.getSize(); if (error(Symbol.getSection(Section))) continue; StringRef Name; @@ -1101,15 +1100,11 @@ void llvm::PrintSymbolTable(const ObjectFile *o) { bool Common = Flags & SymbolRef::SF_Common; bool Hidden = Flags & SymbolRef::SF_Hidden; - if (Common) { - uint32_t Alignment = Symbol.getAlignment(); - Address = Size; - Size = Alignment; - } + if (Common) + Address = Symbol.getSize(); + if (Address == UnknownAddressOrSize) Address = 0; - if (Size == UnknownAddressOrSize) - Size = 0; char GlobLoc = ' '; if (Type != SymbolRef::ST_Unknown) GlobLoc = Global ? 'g' : 'l'; @@ -1151,8 +1146,13 @@ void llvm::PrintSymbolTable(const ObjectFile *o) { SectionName = ""; outs() << SectionName; } - outs() << '\t' - << format("%08" PRIx64 " ", Size); + + outs() << '\t'; + if (Common) + outs() << format("%08" PRIx64 " ", Symbol.getAlignment()); + else if (isa(o)) + outs() << format("%08" PRIx64 " ", Symbol.getSize()); + if (Hidden) { outs() << ".hidden "; } -- 2.7.4