[PATCH] D54999: [clangd] Populate include graph during static indexing action.

Kadir Cetinkaya via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 29 02:48:33 PST 2018


kadircet added inline comments.


================
Comment at: unittests/clangd/IndexActionTests.cpp:28
+std::string PathToURI(llvm::StringRef Path) {
+  return URI::create(Path).toString();
+}
----------------
ilya-biryukov wrote:
> Maybe inline this? This looks simple enough.
yeah but repeating all of that over and over again looks really ugly.


Repository:
  rCTE Clang Tools Extra

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

https://reviews.llvm.org/D54999





More information about the cfe-commits mailing list