diff options
| author | Nico Weber <thakis@chromium.org> | 2021-05-17 20:53:55 -0400 |
|---|---|---|
| committer | Nico Weber <thakis@chromium.org> | 2021-05-18 07:42:58 -0400 |
| commit | 095c520fb436103c9800de0ac15fed83a86f3a43 (patch) | |
| tree | 10b14cb99ee101207805b3173e9ecd61b6953bfa /llvm/lib/Bitcode/Reader/BitcodeReader.cpp | |
| parent | fff84d3a2e661b0de19a2cfbb17fc19eca148830 (diff) | |
[lld/mac] Propagate -(un)exported_symbol(s_list) to privateExtern in Driver
That way, it's done only once instead of every time shouldExportSymbol() is
called.
Possibly a bit faster:
% ministat at_main at_symtodo
x at_main
+ at_symtodo
N Min Max Median Avg Stddev
x 30 3.9732189 4.114846 4.024621 4.0304692 0.037022865
+ 30 3.93766 4.0510042 3.9973931 3.991469 0.028472565
Difference at 95.0% confidence
-0.0390002 +/- 0.0170714
-0.967635% +/- 0.423559%
(Student's t, pooled s = 0.0330256)
In other runs with n=30 it makes no perf difference, so maybe it's just noise.
But being able to quickly and conveniently answer "is this symbol exported?"
is useful for fixing PR50373 and for implementing -dead_strip, so this seems
like a good change regardless.
No behavior change.
Differential Revision: https://reviews.llvm.org/D102661
Diffstat (limited to 'llvm/lib/Bitcode/Reader/BitcodeReader.cpp')
0 files changed, 0 insertions, 0 deletions
