[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