kadircet added inline comments.
================
Comment at: clang-tools-extra/clangd/index/Background.cpp:490
+ auto FS = FSProvider.getFileSystem();
+ llvm::DenseSet<PathRef> TUsToIndex;
+ for (auto &SI : Result.Shards) {
----------------
sammccall wrote:
> nit: again, this patch is adding Path/PathRef aliases to parts of the code
> that don't use them
I believe they represent the intention better then `string/StringRef`that was
the reason I was using them.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D64712/new/
https://reviews.llvm.org/D64712
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
