diff options
| author | Justin Lebar <jlebar@google.com> | 2016-06-30 18:12:25 +0000 |
|---|---|---|
| committer | Justin Lebar <jlebar@google.com> | 2016-06-30 18:12:25 +0000 |
| commit | 8b08d2c5c87bbb0cca4ee59987499f73e252068b (patch) | |
| tree | c9a320a93855a1d175838b8fbf08b493566e8c69 /clang/lib/Frontend/CompilerInvocation.cpp | |
| parent | 6e181cf34182d7805220a2f26dbce7f98eb51662 (diff) | |
| download | bcm5719-llvm-8b08d2c5c87bbb0cca4ee59987499f73e252068b.tar.gz bcm5719-llvm-8b08d2c5c87bbb0cca4ee59987499f73e252068b.zip | |
Don't instantiate a full host toolchain in ASTMatchersTest.
Summary:
This test was stat()'ing large swaths of /usr/lib hundreds of times, as
every invocation of matchesConditionally*() created a new Linux
toolchain.
In addition to being slow, perf indicated this was causing substantial
contention in the kernel.
Something is...interesting in the kernel, as without this patch I
sometimes see ~11m spent in the kernel, and sometimes ~5m. This
corresponds to bimodal ninja check-clang times of ~30s and ~20s.
It's not clear to me exactly what causes the bimodality. In any case,
this change makes this test run in 2.5s, down from 17s, and it seems to
cause us to get the 20s ninja check-clang time unconditionally.
Reviewers: chandlerc
Subscribers: cfe-commits, klimek
Differential Revision: http://reviews.llvm.org/D21810
llvm-svn: 274257
Diffstat (limited to 'clang/lib/Frontend/CompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions
