[clangd] Wait for stdlib indexing to finish before destroying the index

bnbarham · bnbarham · commit 7a268b8c7a33 · 2022-12-19T16:15:53.000-08:00
This will need to be fixed upstream, but temporarily fix on
stable/20221013 to get LSP tests passing.
diff --git a/clang-tools-extra/clangd/ClangdServer.cpp b/clang-tools-extra/clangd/ClangdServer.cpp
@@ -233,6 +233,9 @@ ClangdServer::ClangdServer(const GlobalCompilationDatabase &CDB,
 }
 
 ClangdServer::~ClangdServer() {
+  // Wait for stdlib indexing to finish.
+  if (IndexTasks)
+    IndexTasks->wait();
   // Destroying TUScheduler first shuts down request threads that might
   // otherwise access members concurrently.
   // (Nobody can be using TUScheduler because we're on the main thread).

Original file line number	Diff line number	Diff line change
`@@ -233,6 +233,9 @@ ClangdServer::ClangdServer(const GlobalCompilationDatabase &CDB,`
`233`	`233`	`}`
`234`	`234`
`235`	`235`	`ClangdServer::~ClangdServer() {`
	`236`	`+ // Wait for stdlib indexing to finish.`
	`237`	`+ if (IndexTasks)`
	`238`	`+ IndexTasks->wait();`
`236`	`239`	`// Destroying TUScheduler first shuts down request threads that might`
`237`	`240`	`// otherwise access members concurrently.`
`238`	`241`	`// (Nobody can be using TUScheduler because we're on the main thread).`