[PATCH] D96613: [lld] Add options to trace all symbols and to trace all symbols originated from a file

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added inline comments.


================
Comment at: lld/ELF/Symbols.h:535
+bool Symbol::needToTraceSymbol() {
+  return traced || config->traceAllSymbols ||
+         (this->file != nullptr &&
----------------
MaskRay wrote:
> Can you measure how much slowdown this will cause?
And ping on this question: the condition is pretty complex. Does it affect symbol resolution time?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96613