[PATCH] D55062: [clangd] Partition include graph on auto-index.

Kadir Cetinkaya via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 3 01:47:58 PST 2018


kadircet added inline comments.


================
Comment at: clangd/index/Background.cpp:260
+    std::unique_ptr<IncludeGraph> IG =
+        Index.Sources ? llvm::make_unique<IncludeGraph>(getSubGraph(
+                            URI::create(Path), Index.Sources.getValue()))
----------------
ilya-biryukov wrote:
> What are the cases when it happens in practice? Do we ever run the indexer without producing the include graph?
You are right that's not the case any more.


Repository:
  rCTE Clang Tools Extra

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

https://reviews.llvm.org/D55062





More information about the cfe-commits mailing list