[PATCH] D62381: Use SymbolTable::insert() to implement --trace.

[George Rimar via Phabricator via llvm-commits]

grimar added inline comments.


================
Comment at: lld/ELF/Driver.cpp:1403
   // or DSOs, so we conservatively mark them as address-significant.
   for (Symbol *S : Symtab->getSymbols())
+    if (!S->isPlaceholder() && S->includeInDynsym())
----------------
Should we filter out placeholder symbols returned by `getSymbols` somehow instead?


================
Comment at: lld/ELF/Writer.cpp:1655
       continue;
+
     if (In.SymTab)
----------------
Combine into single `if`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D62381