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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
