[PATCH] D52796: [clangd] Simplify Dex query tree logic and fix missing-posting-list bug
Sam McCall via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Oct 4 10:19:32 PDT 2018
sammccall added inline comments.
================
Comment at: clangd/index/dex/Dex.cpp:184
+ ScopeIterators.push_back(iterator(Token(Token::Kind::Scope, Scope)));
+ if (Req.AnyScope || /*legacy*/ Req.Scopes.empty())
ScopeIterators.push_back(
----------------
ilya-biryukov wrote:
> Why do we still need to push `all()` with a trivial boost for `Scope.empty()`? Is this a legacy way to specify "all scopes"?
Yeah. This is specified in FuzzyFindRequest, with a FIXME to remove as we have explicit AnyScope (but we still have code relying on it for now).
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D52796
More information about the cfe-commits
mailing list