[PATCH] D83099: [clangd] Store index in '.cache/clangd/index' instead of '.clangd/index'

[Kadir Cetinkaya via Phabricator via cfe-commits]

kadircet added inline comments.


================
Comment at: .gitignore:57
+# clangd index. (".clangd" is a config file now, thus trailing slash)
+.clangd/
+.cache
----------------
why do we still need this ? i thought index (and other caches) would reside in `.cache` ?


================
Comment at: clang-tools-extra/clangd/index/Background.h:61
+  // CDBDirectory is the first directory containing a CDB in parent directories
+  // of a file, or user's home directory if none was found, e.g. stdlib headers.
   static Factory createDiskBackedStorageFactory(
----------------
nit: maybe talk about XDG_CACHE instead of home directory as fallback location.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83099