[PATCH] D41991: [clangd] Always use preamble (even stale) for code completion

[Sam McCall via Phabricator via cfe-commits]

sammccall accepted this revision.
sammccall added a comment.

Thanks, LG!



================
Comment at: clangd/Compiler.h:39
+/// Returns null on errors. When non-null value is returned, it is expected to
+/// be consumed by FrontendAction::BeginSourceFile to properly destroy buffer
+/// for \p MainFile.
----------------
nit: "buffer for MainFile" -> "MainFile"
(since MainFile is already a buffer. also: less wrapping!)


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D41991