[PATCH] D63588: [llvm-nm] Generalize symbol types 'N' and 'n'
Fangrui Song via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jun 19 23:58:10 PDT 2019
MaskRay created this revision.
MaskRay added reviewers: alexshap, grimar, jhenderson, rupprecht.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.
Repository:
rL LLVM
https://reviews.llvm.org/D63588
Files:
test/tools/llvm-nm/nonalloc.test
tools/llvm-nm/llvm-nm.cpp
Index: tools/llvm-nm/llvm-nm.cpp
===================================================================
--- tools/llvm-nm/llvm-nm.cpp
+++ tools/llvm-nm/llvm-nm.cpp
@@ -907,22 +907,17 @@
return 'b';
if (Flags & ELF::SHF_ALLOC)
return Flags & ELF::SHF_WRITE ? 'd' : 'r';
- }
- if (SymI->getELFType() == ELF::STT_SECTION) {
- Expected<StringRef> Name = SymI->getName();
- if (!Name) {
- consumeError(Name.takeError());
+ StringRef SecName;
+ if (SecI->getName(SecName))
return '?';
- }
- return StringSwitch<char>(*Name)
- .StartsWith(".debug", 'N')
- .StartsWith(".note", 'n')
- .StartsWith(".comment", 'n')
- .Default('?');
+ if (SecName.startswith(".debug"))
+ return 'N';
+ if (!(Flags & ELF::SHF_WRITE))
+ return 'n';
}
- return 'n';
+ return '?';
}
static char getSymbolNMTypeChar(COFFObjectFile &Obj, symbol_iterator I) {
Index: test/tools/llvm-nm/nonalloc.test
===================================================================
--- test/tools/llvm-nm/nonalloc.test
+++ test/tools/llvm-nm/nonalloc.test
@@ -1,7 +1,9 @@
# RUN: yaml2obj %s -o %t
# RUN: llvm-nm --no-sort %t | FileCheck %s
-# CHECK: n debug_info_main
+# CHECK: N debug_info_main
+# CHECK: n readonly
+# CHECK: ? writable
!ELF
FileHeader:
@@ -12,6 +14,15 @@
Sections:
- Name: .debug_info
Type: SHT_PROGBITS
+ - Name: .readonly
+ Type: SHT_PROGBITS
+ - Name: .writable
+ Type: SHT_PROGBITS
+ Flags: [SHF_WRITE]
Symbols:
- Name: debug_info_main
Section: .debug_info
+ - Name: readonly
+ Section: .readonly
+ - Name: writable
+ Section: .writable
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D63588.205741.patch
Type: text/x-patch
Size: 1671 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190620/0f52bfa6/attachment.bin>
More information about the llvm-commits
mailing list