[PATCH] D98545: [lld-macho][nfc] Give every SyntheticSection a fake InputSection

Jez Ng via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 12 13:03:17 PST 2021


int3 marked 2 inline comments as done.
int3 added inline comments.


================
Comment at: lld/MachO/SyntheticSections.cpp:770-771
     if (auto *defined = dyn_cast<Defined>(sym)) {
+      if (defined->linkerInternal)
+        continue;
       assert(defined->isExternal());
----------------
oontvoo wrote:
> This is true for dso_handle, but probably not for the mh_* symbols?
Indeed. That's why `addSynthetic` takes `linkerInternal` as a parameter -- so that `mh_*` symbols can set it to false :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98545



More information about the llvm-commits mailing list