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 /flang/lib/Frontend/CompilerInvocation.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 'flang/lib/Frontend/CompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions