nridge added a comment.

In D89297#2326838 <https://reviews.llvm.org/D89297#2326838>, @nridge wrote:

> In principle, it should be possible to make the same interface work with 
> `FileIndex` under the hood, but I haven't figured out how. (Specifically, I 
> haven't figured out how to get `FileIndex` to resolve `#include`s, because 
> you can't pass a `MockFS` to it the way you can to `BackgroundIndex`.)

I should mention for completeness that the way we did tests like this in 
Eclipse CDT is... we actually wrote the test files to disk :) But I'm guessing, 
given the efforts that have been made to use `MockFS` in the test suite so far, 
that we don't want to do this for clangd.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89297

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

Reply via email to