[PATCH] D52517: [clangd] clangd-indexer: Drop support for MR-via-YAML

2018-10-04 Thread Sam McCall via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL343759: [clangd] clangd-indexer: Drop support for MR-via-YAML (authored by sammccall, committed by ). Herald added a subscriber: llvm-commits. Repository: rL LLVM https://reviews.llvm.org/D52517 Files

[PATCH] D52517: [clangd] clangd-indexer: Drop support for MR-via-YAML

2018-09-26 Thread Kadir Cetinkaya via Phabricator via cfe-commits
kadircet accepted this revision. kadircet added a comment. This revision is now accepted and ready to land. LGTM Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D52517 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lis

[PATCH] D52517: [clangd] clangd-indexer: Drop support for MR-via-YAML

2018-09-25 Thread Sam McCall via Phabricator via cfe-commits
sammccall updated this revision to Diff 167000. sammccall added a comment. Remove unused function. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D52517 Files: clangd/index/Serialization.h clangd/index/YAMLSerialization.cpp clangd/indexer/IndexerMain.cpp Index: clangd/ind

[PATCH] D52517: [clangd] clangd-indexer: Drop support for MR-via-YAML

2018-09-25 Thread Sam McCall via Phabricator via cfe-commits
sammccall added a comment. This was previously in https://reviews.llvm.org/D52465, but I reduced the scope of that patch to make it easier to land. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D52517 ___ cfe-commits mailing list c

[PATCH] D52517: [clangd] clangd-indexer: Drop support for MR-via-YAML

2018-09-25 Thread Sam McCall via Phabricator via cfe-commits
sammccall created this revision. sammccall added a reviewer: kadircet. Herald added subscribers: cfe-commits, arphaman, jkorous, MaskRay, ioeric, ilya-biryukov. It's slow, and the open-source reduce implementation doesn't scale properly. While here, tidy up some dead headers and comments. Repos