[PATCH] D55650: [clangd] Fix an assertion failure in background index.

Fri Dec 14 03:14:03 PST 2018

hokein marked an inline comment as done.
hokein added inline comments.

================
Comment at: clangd/index/Background.cpp:384
   Action->EndSourceFile();
+  if (Clang->hasDiagnostics() &&
+      Clang->getDiagnostics().hasUncompilableErrorOccurred()) {
----------------
kadircet wrote:
> Then don't need to keep these checks anymore in the endsourcefileaction of indexingaction right?
We need indeed. We have other clients (`clangd-indexer`, and our internal indexer) using the `IndexingAction` -- we don't want symbols from broken TUs because they causes many troublesomes.

Repository:
  rCTE Clang Tools Extra

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

https://reviews.llvm.org/D55650