[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