diff options
| author | Kadir Cetinkaya <kadircet@google.com> | 2019-07-19 13:40:30 +0000 |
|---|---|---|
| committer | Kadir Cetinkaya <kadircet@google.com> | 2019-07-19 13:40:30 +0000 |
| commit | 9dc0160d267333add1d18017fa987110b8d4bc05 (patch) | |
| tree | d600c52832dfee1143f4edac9864cf5415dab3ca | |
| parent | 5905aae16955993ba17af42c701c0f1e20d67ec0 (diff) | |
| download | bcm5719-llvm-9dc0160d267333add1d18017fa987110b8d4bc05.tar.gz bcm5719-llvm-9dc0160d267333add1d18017fa987110b8d4bc05.zip | |
[clangd] Disable background-index on lit-tests by default
Summary:
Since background-index can perform disk writes, we don't want to turn
it on tests that won't clear it.
Reviewers: sammccall
Subscribers: ilya-biryukov, MaskRay, jkorous, arphaman, cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D64990
llvm-svn: 366575
| -rw-r--r-- | clang-tools-extra/clangd/tool/ClangdMain.cpp | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/clang-tools-extra/clangd/tool/ClangdMain.cpp b/clang-tools-extra/clangd/tool/ClangdMain.cpp index 5db3d5c2306..60f1e424400 100644 --- a/clang-tools-extra/clangd/tool/ClangdMain.cpp +++ b/clang-tools-extra/clangd/tool/ClangdMain.cpp @@ -355,8 +355,12 @@ int main(int argc, char *argv[]) { InputStyle = JSONStreamStyle::Delimited; LogLevel = Logger::Verbose; PrettyPrint = true; + // Disable background index on lit tests by default to prevent disk writes. + if (!EnableBackgroundIndex.getNumOccurrences()) + EnableBackgroundIndex = false; // Ensure background index makes progress. - BackgroundQueue::preventThreadStarvationInTests(); + else if (EnableBackgroundIndex) + BackgroundQueue::preventThreadStarvationInTests(); } if (Test || EnableTestScheme) { static URISchemeRegistry::Add<TestScheme> X( |
