[PATCH] D49417: [clangd] Implement trigram generation algorithm for new symbol index

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 19 02:39:51 PDT 2018


sammccall added a comment.
Herald added a subscriber: arphaman.

In https://reviews.llvm.org/D49417#1166538, @omtcyfz wrote:

> Addressed all comments submitted by Eric.
>
> As discussed internally, I should also exercise my naming skills and come up with a better for the symbol index to substitute "Noctem" which doesn't point to any project's feature.


Ooh, a bikeshed!
Suggestion: `dex` - short, pronouncable, suggests `index`, and is a trigram :-)


https://reviews.llvm.org/D49417





More information about the cfe-commits mailing list