diff options
| author | Alex Richardson <Alexander.Richardson@cl.cam.ac.uk> | 2021-09-20 12:32:56 +0100 |
|---|---|---|
| committer | Alex Richardson <Alexander.Richardson@cl.cam.ac.uk> | 2021-09-20 12:55:56 +0100 |
| commit | 6d7b3d6b3a8dbd62650b6c3dae1fe904a8ae9048 (patch) | |
| tree | b2bb99d8a00e35836c93c14cebb24cd0870c9c47 /clang/lib/Frontend/CreateInvocationFromCommandLine.cpp | |
| parent | 7fc12b822c5d1360780667af94c218733c3fc4e0 (diff) | |
| download | llvm-6d7b3d6b3a8dbd62650b6c3dae1fe904a8ae9048.zip llvm-6d7b3d6b3a8dbd62650b6c3dae1fe904a8ae9048.tar.gz llvm-6d7b3d6b3a8dbd62650b6c3dae1fe904a8ae9048.tar.bz2 | |
Fix CLANG_ENABLE_STATIC_ANALYZER=OFF building all analyzer source
Since https://reviews.llvm.org/D87118, the StaticAnalyzer directory is
added unconditionally. In theory this should not cause the static analyzer
sources to be built unless they are referenced by another target. However,
the clang-cpp target (defined in clang/tools/clang-shlib) uses the
CLANG_STATIC_LIBS global property to determine which libraries need to
be included. To solve this issue, this patch avoids adding libraries to
that property if EXCLUDE_FROM_ALL is set.
In case something like this comes up again: `cmake --graphviz=targets.dot`
is quite useful to see why a target is included as part of `ninja all`.
Reviewed By: thakis
Differential Revision: https://reviews.llvm.org/D109611
Diffstat (limited to 'clang/lib/Frontend/CreateInvocationFromCommandLine.cpp')
0 files changed, 0 insertions, 0 deletions