[PATCH] D39738: [clangd] Sort completion results.

[Francisco Lopes da Silva via Phabricator via cfe-commits]

francisco.lopes added a comment.

OK. I do agree serialization, etc, is a stumbling block on delivering large results fast. I just wanted to warn on this because I've just seen ycmd devs went through this, where sorting was one of the bottlenecks, like, to `std::sort` 43k results instead of simply partial sorting them just over a 30 results window, which is orders of magnitude faster.

If LSP completion ends up only feasible when there's filtering server side, with its own hard-coded sorting/filtering/ranking mechanisms, it's really sad, because it completely reduce flexibility of consumers wishing to apply their own (custom/configurable/uniform) sorting mechanisms.


Repository:
  rL LLVM

https://reviews.llvm.org/D39738