diff options
| author | Fangrui Song <i@maskray.me> | 2023-07-27 20:51:42 -0700 |
|---|---|---|
| committer | Fangrui Song <i@maskray.me> | 2023-07-27 20:51:42 -0700 |
| commit | 9ea44c6894270546d6d88ef0d3abcf1e1876acae (patch) | |
| tree | 80c6e846a5ed2f0650124df114735919644d15f6 /llvm/lib/Bitcode/Reader/BitcodeAnalyzer.cpp | |
| parent | 74b6b113f4a1c98f8859a08aad595b91aa03d197 (diff) | |
| download | llvm-9ea44c6894270546d6d88ef0d3abcf1e1876acae.zip llvm-9ea44c6894270546d6d88ef0d3abcf1e1876acae.tar.gz llvm-9ea44c6894270546d6d88ef0d3abcf1e1876acae.tar.bz2 | |
[llvm-objdump] -d: don't display mapping symbols as labels
Similar to D96617 for llvm-symbolizer.
This patch matches the GNU objdump -d behavior to suppress printing
labels for mapping symbols. Mapping symbol names don't convey much
information.
When --show-all-symbols (not in GNU) is specified, we still print
mapping symbols.
Note: the `for (size_t SI = 0, SE = Symbols.size(); SI != SE;)` loops
needs to iterate all mapping symbols, even if they are not displayed.
We use the new field `IsMappingSymbol` to recognize mapping symbols.
This field also enables simplification after D139131.
ELF/ARM/disassemble-all-mapping-symbols.s is enhanced to add `.space 2`.
If `End = std::min(End, Symbols[SI].Addr);` is not correctly set, we
would print a `.word`.
Reviewed By: jhenderson, jobnoorman, peter.smith
Differential Revision: https://reviews.llvm.org/D156190
Diffstat (limited to 'llvm/lib/Bitcode/Reader/BitcodeAnalyzer.cpp')
0 files changed, 0 insertions, 0 deletions