[PATCH] D53433: [clangd] auto-index stores symbols per-file instead of per-TU.

Kadir Cetinkaya via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 1 09:09:28 PDT 2018


kadircet added inline comments.


================
Comment at: clangd/index/Background.cpp:235
+    IndexedFileDigests[Path] = FilesToUpdate.lookup(Path);
+    IndexedSymbols.update(Path,
+                          make_unique<SymbolSlab>(std::move(Syms).build()),
----------------
This call is already thread-safe, no need for it to be under lock.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D53433





More information about the cfe-commits mailing list