diff options
| author | Nico Weber <thakis@chromium.org> | 2021-05-17 18:17:18 -0400 |
|---|---|---|
| committer | Nico Weber <thakis@chromium.org> | 2021-05-17 20:04:45 -0400 |
| commit | bc588f9961117d5705e97eb2191e901e92fe9094 (patch) | |
| tree | 6846dd9a49651a5ce13c2ac1cb4b1032b2ec3489 /clang/lib/CodeGen/CodeGenModule.cpp | |
| parent | 6d3e3ae8a9ca10e063d541a959f4fe4cdb003dba (diff) | |
[lld/mac] Inline a check
`match()` can only return for non-empty vectors, but at least in
non-LTO builds that isn't clear to the compiler. Help it out.
This is a minor but measurable speedup on my machine (but less
than what we might've lost in https://reviews.llvm.org/D100818#2764272 --
bot note higher N on this measurement here, so higher confidence here):
% ministat at_main at_branch
x at_main
+ at_branch
N Min Max Median Avg Stddev
x 30 3.9243979 4.0395119 3.987375 3.9826236 0.027567796
+ 30 3.8495831 4.0009291 3.931325 3.9347135 0.037832878
Difference at 95.0% confidence
-0.0479101 +/- 0.0171102
-1.20298% +/- 0.429622%
(Student's t, pooled s = 0.0331007)
No behavior change.
Eventually we should apply these lists at symbol parse time instead of
every time shouldExportSymbol() though :)
Differential Revision: https://reviews.llvm.org/D102655
Diffstat (limited to 'clang/lib/CodeGen/CodeGenModule.cpp')
0 files changed, 0 insertions, 0 deletions
