This revision was automatically updated to reflect the committed changes.
Closed by commit rCTE331457: [clangd] Incorporate #occurrences in scoring code
complete results. (authored by sammccall, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D46183?vs=145021&id=145024#toc
Re
ilya-biryukov accepted this revision.
ilya-biryukov added a comment.
This revision is now accepted and ready to land.
LGTM
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D46183
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
sammccall updated this revision to Diff 145021.
sammccall added a comment.
Fix tests.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D46183
Files:
clangd/CodeComplete.cpp
clangd/index/Index.cpp
clangd/index/Index.h
clangd/index/MemIndex.cpp
unittests/clangd/CodeComplet
sammccall added a comment.
Added a test - it's kind of perfunctory, I really want to organize symbol
scoring more thoroughly which will provide a better place to test well.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D46183
___
c
sammccall updated this revision to Diff 145011.
sammccall added a comment.
Add completion test.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D46183
Files:
clangd/CodeComplete.cpp
clangd/index/Index.cpp
clangd/index/Index.h
clangd/index/MemIndex.cpp
unittests/clangd/C
ilya-biryukov added a comment.
the patch LG and fixes the problem of llvm namespace not showing up in
completions from the static index.
Let's add some tests and ship it!
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D46183
___
cfe
sammccall created this revision.
sammccall added a reviewer: ilya-biryukov.
Herald added subscribers: cfe-commits, jkorous, MaskRay, ioeric, klimek.
needs tests
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D46183
Files:
clangd/CodeComplete.cpp
clangd/index/Index.cpp
clan
