[PATCH] D81066: [clangd] Populate the parse options to CodeCompletion/SignatureHelp.

[Sam McCall via Phabricator via cfe-commits]

sammccall accepted this revision.
sammccall added inline comments.
This revision is now accepted and ready to land.


================
Comment at: clang-tools-extra/clangd/CodeComplete.cpp:1108
+      std::move(CI),
+      !CompletingInPreamble ? &(Input.Preamble.Preamble) : nullptr,
       std::move(ContentsBuffer), std::move(VFS), IgnoreDiags);
----------------
this change looks suspicious, really needed?


================
Comment at: clang-tools-extra/clangd/Compiler.h:57
   const SymbolIndex *Index = nullptr;
-  ParseOptions Opts;
+  ParseOptions Opts = ParseOptions();
 };
----------------
this is the default already - only need to do this for scalar types that would otherwise have indeterminate default


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81066