[PATCH] D81456: [clangd] Get rid of WantDiagnostics::Yes

[Kadir Cetinkaya via Phabricator via cfe-commits]

kadircet planned changes to this revision.
kadircet marked an inline comment as done.
kadircet added inline comments.


================
Comment at: clang-tools-extra/clangd/ClangdLSPServer.cpp:670
   auto Version = DraftMgr.addDraft(File, Params.textDocument.version, Contents);
-  Server->addDocument(File, Contents, encodeVersion(Version),
-                      WantDiagnostics::Yes);
+  Server->addDocument(File, Contents, encodeVersion(Version));
 }
----------------
sammccall wrote:
> This cuts against the assertion in the description that there are no uses!
> 
> And I believe the debounce test demonstrates that that we will in fact debounce an initial request that's `Auto`, right?
> So isn't this going to add 0.5sec to time-before-first-diagnostics in all cases?
ah right, let's reconsider this after having a special initial AST build (one without preamble serialization/deserialization and clang-tidy) to reduce first diagnostic latency.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D81456/new/

https://reviews.llvm.org/D81456