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 /clang/unittests/Frontend/CompilerInvocationTest.cpp | |
parent | fff84d3a2e661b0de19a2cfbb17fc19eca148830 (diff) | |
download | llvm-095c520fb436103c9800de0ac15fed83a86f3a43.zip llvm-095c520fb436103c9800de0ac15fed83a86f3a43.tar.gz llvm-095c520fb436103c9800de0ac15fed83a86f3a43.tar.bz2 |
[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 'clang/unittests/Frontend/CompilerInvocationTest.cpp')
0 files changed, 0 insertions, 0 deletions