[PATCH] D49991: [clangd] Do not build AST if no diagnostics were requested

2018-07-31 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL338378: [clangd] Do not build AST if no diagnostics were requested (authored by ibiryukov, committed by ). Herald added a subscriber: llvm-commits. Repository: rL LLVM https://reviews.llvm.org/D49991

[PATCH] D49991: [clangd] Do not build AST if no diagnostics were requested

2018-07-31 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov updated this revision to Diff 158234. ilya-biryukov added a comment. - Rebase onto head Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D49991 Files: clangd/TUScheduler.cpp Index: clangd/TUScheduler.cpp ===

[PATCH] D49991: [clangd] Do not build AST if no diagnostics were requested

2018-07-31 Thread Eric Liu via Phabricator via cfe-commits
ioeric accepted this revision. ioeric added a comment. This revision is now accepted and ready to land. lgtm Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D49991 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.l

[PATCH] D49991: [clangd] Do not build AST if no diagnostics were requested

2018-07-30 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov created this revision. ilya-biryukov added a reviewer: ioeric. Herald added subscribers: arphaman, jkorous, MaskRay, javed.absar. It can be washed out from the cache before first access anyway, so building it can just be a waste of time. Repository: rCTE Clang Tools Extra https: