[PATCH] D39882: [clangd] Filter completion results by fuzzy-matching identifiers.

Ilya Biryukov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 22 07:59:49 PST 2017


ilya-biryukov added a comment.

We definitely need to:

- Rebase this change on top of current head (to account for limits and scoring)
- Set `incomplete=true` for fuzzy-matched completion results

Maybe also make fuzzy-matching configurable via a flag? Off-by-default for now, so we could start testing it before we finish optimizing single-identifier edits. When we have it, enable fuzzy-matching by default.



================
Comment at: clangd/ClangdUnit.cpp:427
+  static bool fuzzyMatch(StringRef Filter, StringRef Target) {
+    llvm::errs() << "match " << Target << " against " << Filter << "\n";
+    size_t TPos = 0;
----------------
Debug output sneaked into the commit.


https://reviews.llvm.org/D39882





More information about the cfe-commits mailing list