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

[Greg Clayton via Phabricator via llvm-commits]

clayborg 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});
----------------
int3 wrote:
> 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?
No I am not. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89285