ilya-biryukov added inline comments.
================ Comment at: unittests/clangd/ClangdTests.cpp:941 -TEST_F(ClangdVFSTest, InsertIncludes) { - MockFSProvider FS; ---------------- Do we test the same thing somewhere else (e.g. code completion) in one of the dependent changes? Maybe it's worth noting in the commit description that this test was not removed completely, but instead moved to <new-test-name-here>? Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D46676 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits