[PATCH] D78888: [llvm-objcopy][MachO] Fix symbol table
Shoaib Meenai via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 27 15:39:42 PDT 2020
smeenai added inline comments.
================
Comment at: llvm/tools/llvm-objcopy/MachO/Object.h:112
+
+ bool isExternalSymbol() const { return !isSTAB() && n_type & MachO::N_EXT; }
----------------
alexshap wrote:
> smeenai wrote:
> > Why do we need the special case for STAB symbols?
> according to https://developer.apple.com/documentation/kernel/nlist_64/1583944-n_type?language=objc
> if this symbol is a STAB then the entire n_type value is interpreted differently
Ah, right, that makes sense.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78888/new/
https://reviews.llvm.org/D78888
More information about the llvm-commits
mailing list