[PATCH] D36398: [clangd] Check if CompileCommand has changed on forceReparse.

Manuel Klimek via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 7 08:01:13 PDT 2017


klimek added inline comments.


================
Comment at: clangd/ClangdUnitStore.cpp:45
+             .first;
+    Result.RemovedFile = nullptr;
+  } else if (!compileCommandsAreEqual(It->second->getCompileCommand(),
----------------
Just say RemovedFile = nullptr in the struct?


================
Comment at: clangd/ClangdUnitStore.h:45-48
+  struct RecreateResult {
+    std::shared_ptr<CppFile> FileInCollection;
+    std::shared_ptr<CppFile> RemovedFile;
+  };
----------------
Not obvious to me what things in there mean.


https://reviews.llvm.org/D36398





More information about the cfe-commits mailing list