[clangd] Skip (most) predefined macros when scanning for preamble patching.
authorSam McCall <sam.mccall@gmail.com>
Sat, 7 May 2022 21:40:27 +0000 (23:40 +0200)
committerSam McCall <sam.mccall@gmail.com>
Mon, 9 May 2022 13:33:31 +0000 (15:33 +0200)
commitf1a9c4b717be9a9a730f837dfd70df69d1daf44f
tree6b924b816c30e086ac8209f6d1eed58bb6d3fec3
parenta425cac31e2e4cee8e14b7b9a99c8ba17c1ebb52
[clangd] Skip (most) predefined macros when scanning for preamble patching.

This is unneccesary work.
With this change, we skip generating and lexing ~10k of predefines twice.

A dumb benchmark of building a preamble for an empty file in a loop shows:
 - before: 1.90ms/run
 - after: 1.36ms/run
So this should be worth 0.5ms for each AST build and code completion.

There can be a functional difference, but it's very minor.
If the preamble contains e.g. `#ifndef __llvm__ ... #endif` then before we would
not take it. After this change we will take the branch (single-file mode takes
all branches with unknown conditions) and so gather different directives.

However I think this is negligible:
 - this is already true of non-builtin macros (from included headers).
   We've had no complaints.
 - this affects the baseline and modified in the same way, so only makes a
   difference transiently when code guarded by such an #ifdef is being edited

Differential Revision: https://reviews.llvm.org/D125179
clang-tools-extra/clangd/Preamble.cpp