[PATCH] D97027: [llvm-objdump][MachO] Add support for dumping function starts

[Nico Weber via Phabricator via llvm-commits]

thakis accepted this revision.
thakis added a comment.
This revision is now accepted and ready to land.

lg, but:



================
Comment at: llvm/tools/llvm-objdump/MachODump.cpp:1141
+  for (uint64_t S : FunctionStarts) {
+    uint64_t Addr = BaseSegmentAddress + S;
+    if (O->is64Bit())
----------------
Should this add the base segment address? I'd expect `--function-starts` to dump the contents of FUNCTION_STARTS unmodified.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D97027/new/

https://reviews.llvm.org/D97027