[PATCH] D48375: [clangd] Remove FilterText from the index.

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 20 09:04:26 PDT 2018


sammccall created this revision.
sammccall added a reviewer: ioeric.
Herald added subscribers: cfe-commits, jkorous, MaskRay, ilya-biryukov.

It's almost always identical to Name, and in fact we never used it (we used name
instead).
The only case where they differ is objc method selectors (foo: vs foo:bar:).
We can live with the latter for both name and filterText, so I've made that
change too.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D48375

Files:
  clangd/CodeComplete.cpp
  clangd/CodeCompletionStrings.cpp
  clangd/CodeCompletionStrings.h
  clangd/index/Index.cpp
  clangd/index/Index.h
  clangd/index/Merge.cpp
  clangd/index/SymbolCollector.cpp
  clangd/index/SymbolYAML.cpp
  unittests/clangd/CodeCompletionStringsTests.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D48375.152101.patch
Type: text/x-patch
Size: 5911 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180620/a9ddf9f9/attachment.bin>


More information about the cfe-commits mailing list