njames93 added a comment. In D88172#2414249 <https://reviews.llvm.org/D88172#2414249>, @sammccall wrote:
> @kbobyrev ping... I think we do actually want to land this, for use with > `.clang-tidy` files after D91029 <https://reviews.llvm.org/D91029> Yes, I was looking at copying the config cache for use with `.clang-tidy` ================ Comment at: clang-tools-extra/clangd/support/FileCache.h:43 + // Path must be absolute. + FileCache(llvm::StringRef Path); + ---------------- nit: Is it more descriptive to have this spelled as `PathRef`, same for `path()` below. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88172/new/ https://reviews.llvm.org/D88172 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits