diff options
author | James Henderson <jh7370@my.bristol.ac.uk> | 2019-01-08 10:58:05 +0000 |
---|---|---|
committer | James Henderson <jh7370@my.bristol.ac.uk> | 2019-01-08 10:58:05 +0000 |
commit | 6135b0f88c83a480d7c2116ac9c0e589f536e0c4 (patch) | |
tree | 40d207b0ecfe3a6db6a291d118bcece351b1f58e /llvm/tools/llvm-readobj/ELFDumper.cpp | |
parent | 4ba1777a2da7a0104baf63a2f28005577039324b (diff) | |
download | bcm5719-llvm-6135b0f88c83a480d7c2116ac9c0e589f536e0c4.tar.gz bcm5719-llvm-6135b0f88c83a480d7c2116ac9c0e589f536e0c4.zip |
[llvm-readobj] Don't print '@' at end of unversioned dynsym names
This fixes https://bugs.llvm.org/show_bug.cgi?id=40097. The problem was
caused by a regression in r188022.
See also r350614.
Reviewed by: rupprecht, mstorsjo, Higuoxing, jakehehrlich
Differential Revision: https://reviews.llvm.org/D56319
llvm-svn: 350615
Diffstat (limited to 'llvm/tools/llvm-readobj/ELFDumper.cpp')
-rw-r--r-- | llvm/tools/llvm-readobj/ELFDumper.cpp | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/llvm/tools/llvm-readobj/ELFDumper.cpp b/llvm/tools/llvm-readobj/ELFDumper.cpp index 6b69f7e844e..93254717e92 100644 --- a/llvm/tools/llvm-readobj/ELFDumper.cpp +++ b/llvm/tools/llvm-readobj/ELFDumper.cpp @@ -818,8 +818,10 @@ std::string ELFDumper<ELFT>::getFullSymbolName(const Elf_Sym *Symbol, bool IsDefault; StringRef Version = getSymbolVersion(StrTable, &*Symbol, IsDefault); - FullSymbolName += (IsDefault ? "@@" : "@"); - FullSymbolName += Version; + if (!Version.empty()) { + FullSymbolName += (IsDefault ? "@@" : "@"); + FullSymbolName += Version; + } return FullSymbolName; } |