[PATCH] D92366: [lld-macho] Flesh out STABS implementation

[Greg Clayton via Phabricator via llvm-commits]

clayborg added inline comments.


================
Comment at: lld/MachO/SyntheticSections.cpp:654-655
+  for (Defined *defined : symbolsNeedingStabs) {
+    if (defined->isAbsolute())
+      continue;
+
----------------
You shouldn't need this because of lines 634 and 635 skips adding absolute symbols to symbolsNeedingStabs


================
Comment at: lld/MachO/SyntheticSections.cpp:677
+    // symbols?
+    if (isec->name == "__text") {
+      symStab.type = MachO::N_FUN;
----------------
Is this going to always work for all functions? This really needs to be bullet proof or dsymutil and lldb won't be able to find functions for this symbol.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92366