summaryrefslogtreecommitdiffstats
path: root/llvm/tools/llvm-nm/llvm-nm.cpp
diff options
context:
space:
mode:
authorSunil Srivastava <sunil_srivastava@playstation.sony.com>2019-02-03 22:40:01 +0000
committerSunil Srivastava <sunil_srivastava@playstation.sony.com>2019-02-03 22:40:01 +0000
commitb33b410283e2759797b5ed264a4c6889a80fc5bb (patch)
treee6c890808ff471e596f3bde01ace18f50ac14d49 /llvm/tools/llvm-nm/llvm-nm.cpp
parent6ccad0a7d03b672e8653dfd4722ac10e26d3901f (diff)
downloadbcm5719-llvm-b33b410283e2759797b5ed264a4c6889a80fc5bb.tar.gz
bcm5719-llvm-b33b410283e2759797b5ed264a4c6889a80fc5bb.zip
Compute the correct symbol size in llvm-nm even without --print-size
In llvm-nm, the symbol size was being computed only with --print-size option, even though it was being printed in other cases, such as with --format=posix. This patch simply removes the guard, so that the size is computed independently of the later decision to print it or not. Fixes PR39997. Differential Revision: https://reviews.llvm.org/D57599 llvm-svn: 353011
Diffstat (limited to 'llvm/tools/llvm-nm/llvm-nm.cpp')
-rw-r--r--llvm/tools/llvm-nm/llvm-nm.cpp6
1 files changed, 2 insertions, 4 deletions
diff --git a/llvm/tools/llvm-nm/llvm-nm.cpp b/llvm/tools/llvm-nm/llvm-nm.cpp
index 3a83fdb1424..7c009297139 100644
--- a/llvm/tools/llvm-nm/llvm-nm.cpp
+++ b/llvm/tools/llvm-nm/llvm-nm.cpp
@@ -1190,10 +1190,8 @@ dumpSymbolNamesFromObject(SymbolicFile &Obj, bool printName,
NMSymbol S = {};
S.Size = 0;
S.Address = 0;
- if (PrintSize) {
- if (isa<ELFObjectFileBase>(&Obj))
- S.Size = ELFSymbolRef(Sym).getSize();
- }
+ if (isa<ELFObjectFileBase>(&Obj))
+ S.Size = ELFSymbolRef(Sym).getSize();
if (PrintAddress && isa<ObjectFile>(Obj)) {
SymbolRef SymRef(Sym);
Expected<uint64_t> AddressOrErr = SymRef.getAddress();
OpenPOWER on IntegriCloud