sammccall marked an inline comment as done.
sammccall added inline comments.
================
Comment at: clang-tools-extra/clangd/ClangdServer.cpp:184
+ TFS, CDB,
BackgroundIndexStorage::createDiskBackedStorageFactory(
[&CDB](llvm::StringRef File) { return CDB.getProjectInfo(File); }),
----------------
kadircet wrote:
> I think we should move storage factory into options too ?
The storage factory isn't optional, nor does it have a good default. So it
seems mechanically awkward to put it in the struct, and I'm not sure there's
much benefit...
How would you see this being initialized?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D83157/new/
https://reviews.llvm.org/D83157
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
