[PATCH] D59811: [clangd] Add fallback mode for code completion when compile command or preamble is not ready.

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 8 06:45:33 PDT 2019


sammccall added inline comments.


================
Comment at: unittests/clangd/ClangdTests.cpp:1082
+  EXPECT_EQ(Res.Context, CodeCompletionContext::CCC_Recovery);
+  ASSERT_TRUE(Server.blockUntilIdleForTest());
+  EXPECT_THAT(cantFail(runCodeComplete(Server, FooCpp, Code.point(),
----------------
sammccall wrote:
> do I understand right that this test is currently racy?
Sorry, missed your comment. Test LG


================
Comment at: unittests/clangd/ClangdTests.cpp:1080
+  // This will make compile command broken and preamble absent.
+  CDB.ExtraClangFlags = {"yolo.cc"};
+  Server.addDocument(FooCpp, Code.code());
----------------
(`-this-flag-does-not-exist` might be clearer)


Repository:
  rCTE Clang Tools Extra

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

https://reviews.llvm.org/D59811





More information about the cfe-commits mailing list