[PATCH] D89285: [lld-macho] Emit local symbols in symtab; record metadata in LC_DYSYMTAB

[Jez Ng via Phabricator via llvm-commits]

int3 added inline comments.


================
Comment at: lld/MachO/SyntheticSections.cpp:643
+          if (!defined->isExternal()) {
+            uint32_t strx = stringTableSection.addString(sym->getName());
+            localSymbols.push_back({sym, strx});
----------------
clayborg wrote:
> Do all symbols have names? If not we should check that the symbol has a name and not add it if it doesn't?
To my knowledge, yes they do... are you aware of a way to generate unnamed local symbols?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89285