diff options
| author | Sam McCall <sam.mccall@gmail.com> | 2022-07-13 21:11:44 +0200 |
|---|---|---|
| committer | Sam McCall <sam.mccall@gmail.com> | 2022-09-07 16:18:31 +0200 |
| commit | 897f3ddc6154fee9bc072f800ceab8202b236b0a (patch) | |
| tree | c514798e91b70f8c5f42399d8c262dac7018bd56 /clang/lib/CodeGen/CodeGenFunction.cpp | |
| parent | fb9fc79809d5c2c3ab8809f7ff98fbfa8c3a9e7e (diff) | |
| download | llvm-897f3ddc6154fee9bc072f800ceab8202b236b0a.zip llvm-897f3ddc6154fee9bc072f800ceab8202b236b0a.tar.gz llvm-897f3ddc6154fee9bc072f800ceab8202b236b0a.tar.bz2 | |
[Sema] Move Diags.isIgnored() checks off hot paths, it's not free. NFC
This speeds up clangd's buildAST() (i.e. parsing with a preamble) by 5% on
clangd/AST.cpp, by avoiding filling up the diagnostic state map with entries for
all the files where templates are being instantiated from.
(I would assume it has a similar effect on PCH and modules compiles).
This approach is obviously pretty fragile, and we should find ways to make
isIgnored() cheaper instead. But these changes in particular don't seem to make
the code worse in any case.
Differential Revision: https://reviews.llvm.org/D129683
Diffstat (limited to 'clang/lib/CodeGen/CodeGenFunction.cpp')
0 files changed, 0 insertions, 0 deletions