[PATCH] D42669: [clangd] Enable completion index by default, limit results to 100.

2018-01-30 Thread Sam McCall via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL323734: [clangd] Enable completion index by default, limit results to 100. (authored by sammccall, committed by ). Herald added a subscriber: llvm-commits. Repository: rL LLVM

[PATCH] D42669: [clangd] Enable completion index by default, limit results to 100.

2018-01-30 Thread Eric Liu via Phabricator via cfe-commits
ioeric accepted this revision. ioeric added a comment. Woohoo! Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D42669 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D42669: [clangd] Enable completion index by default, limit results to 100.

2018-01-29 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov accepted this revision. ilya-biryukov added a comment. This revision is now accepted and ready to land. LGTM Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D42669 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D42669: [clangd] Enable completion index by default, limit results to 100.

2018-01-29 Thread Sam McCall via Phabricator via cfe-commits
sammccall created this revision. sammccall added reviewers: hokein, ioeric, ilya-biryukov. Herald added subscribers: cfe-commits, jkorous-apple, klimek. This should speed up global code completion by avoiding deserializing preamble declarations to look up names. The tradeoff is memory usage.