[PATCH] D96353: [clangd] Use ML Code completion ranking as default.

[Haojian Wu via Phabricator via cfe-commits]

hokein added inline comments.


================
Comment at: clang-tools-extra/clangd/unittests/CodeCompleteTests.cpp:651
+  auto Results =
+      completions("int main() { abs^ }", {func("absl"), func("absb")});
   EXPECT_THAT(Results.Completions,
----------------
sorry, I didn't infer the motivation of this change func->ns from this patch, could you explain?


================
Comment at: clang-tools-extra/clangd/unittests/CodeCompleteTests.cpp:655
   Results = completions("int main() { abs^ }",
-                        {withReferences(10000, ns("absl")), func("absb")});
-  EXPECT_THAT(Results.Completions,
-              HasSubsequence(Named("absl"), Named("absb")));
+                        {withReferences(100, func("absl")), func("absb")});
 }
----------------
looks like the Results here is not verified, as you remove the line below, is it intentional?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96353