[PATCH] D119050: [llvm-objdump/mac] Add new function starts print mode
Keith Smiley via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Oct 14 14:12:24 PDT 2022
keith added a comment.
New test requires https://reviews.llvm.org/D135988
================
Comment at: llvm/tools/llvm-objdump/MachODump.cpp:1107
+ if (It != SymbolNames.end())
+ outs() << It->second << "\n";
+ } else {
----------------
smeenai wrote:
> So if we can't find the address associated with a symbol, we don't print anything for it? Does that match dyldinfo's behavior? Should we have a test for it?
So turns out that dyldinfo prints a `?` beside the address in this case. I've mirrored that here and added a test, but in the case you only ask for names I stilled opted to print nothing since it doesn't seem like a ton of `?`'s w/o any other info would be very useful output, and if you have _some_ symbols it might distract you from those. Interested to hear thoughts on this since names only not a mode dyldinfo has.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D119050/new/
https://reviews.llvm.org/D119050
More information about the llvm-commits
mailing list