sammccall accepted this revision.
sammccall added a comment.

Thanks, this seems better than crashing.
The practical result isn't wonderful: the two are going to fight over index 
files to some extent. But this can happen with different clangd versions (that 
use different index format versions) anyway. So this is really just graceful 
recovery from a bad situation.

In D87673#2273347 <https://reviews.llvm.org/D87673#2273347>, @ArcsinX wrote:

> Unsure about test for this

Agree, testing this seems hard and not that useful.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87673

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to