[PATCH] D88814: [clangd] Enable partial namespace matches for workspace symbols

Kadir Cetinkaya via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 6 03:31:30 PDT 2020


kadircet updated this revision to Diff 296402.
kadircet marked 6 inline comments as done.
kadircet added a comment.

- Perform sub-scope matching rather than substring match for partial namespaces.
- Apply a penalty for partially matching namespaces.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88814

Files:
  clang-tools-extra/clangd/FindSymbols.cpp
  clang-tools-extra/clangd/unittests/FindSymbolsTests.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D88814.296402.patch
Type: text/x-patch
Size: 6402 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20201006/0797b401/attachment.bin>


More information about the cfe-commits mailing list