[PATCH] D89297: [clangd] Add a TestWorkspace utility
Nathan Ridge via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Oct 15 10:22:46 PDT 2020
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
More information about the cfe-commits
mailing list