[PATCH] D58440: [clangd] Store index in '.clangd/index' instead of '.clangd-index'

Ilya Biryukov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Feb 20 10:59:11 PST 2019


ilya-biryukov added inline comments.


================
Comment at: clang-tools-extra/clangd/index/BackgroundIndexStorage.cpp:75
     llvm::SmallString<128> CDBDirectory(Directory);
-    llvm::sys::path::append(CDBDirectory, ".clangd-index/");
+    llvm::sys::path::append(CDBDirectory, ".clangd", "index/");
     DiskShardRoot = CDBDirectory.str();
----------------
kadircet wrote:
> NIT: it looks like we don't need the "/" at the end of "index/"
Done.
I was not sure if it is important or not. Everything appears to work without it, so removing it now.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D58440





More information about the cfe-commits mailing list