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

Hongtao Yu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Feb 27 22:41:11 PST 2021


hoy added a comment.

@MaskRay I think your concern is primarily about the extra condition checks in `Symbol::needToTraceSymbol` that may slow down the linker. Is there a linker performance testing system on your side that you can help evaluate it? I have manually run the patch against medium-sized programs and haven't seen regressions. Otherwise, I agree with @smeenai that we should be open to changes that make life easier with little cost.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96613



More information about the llvm-commits mailing list