[PATCH] D42429: [clangd] Moved caching of compile commands to ClangdServer

2018-01-25 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rCTE323420: [clangd] Moved caching of compile commands to ClangdServer (authored by ibiryukov, committed by ). Changed prior to commit: https://reviews.llvm.org/D42429?vs=131429&id=131432#toc Repository:

[PATCH] D42429: [clangd] Moved caching of compile commands to ClangdServer

2018-01-25 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov updated this revision to Diff 131429. ilya-biryukov marked 5 inline comments as done. ilya-biryukov added a comment. Herald added subscribers: hintonda, mgorny. - Addressed review comments Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D42429 Files: clangd/CMakeL

[PATCH] D42429: [clangd] Moved caching of compile commands to ClangdServer

2018-01-25 Thread Sam McCall via Phabricator via cfe-commits
sammccall accepted this revision. sammccall added a comment. This revision is now accepted and ready to land. Thanks for this cleanup! The way we deal with ResourceDir still doesn't feel ideal, but that's no worse here and everything else is better! Comment at: clangd/ClangdS

[PATCH] D42429: [clangd] Moved caching of compile commands to ClangdServer

2018-01-23 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov created this revision. ilya-biryukov added a reviewer: sammccall. Herald added subscribers: ioeric, jkorous-apple, klimek. It allows to get rid of CppFile::getLastCommand and simplify the code in the upcoming threading patch. Repository: rCTE Clang Tools Extra https://reviews.ll