Re: [PATCH] D54998: [clangd] Build and test IndexBenchmark in check-clangd

2018-11-28 Thread Tom Weaver via cfe-commits
Hiya Haojian, I've been looking over the build bots recently and believe r347753 may have caused the following test: clangd/index-tools.test to start failing. You can see the failing test in the logs of the following build bot: http://lab.llvm.org:8011/builders/clang-s390x-linux/builds/20851/st

[PATCH] D54998: [clangd] Build and test IndexBenchmark in check-clangd

2018-11-28 Thread Haojian Wu via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rCTE347753: [clangd] Build and test IndexBenchmark in check-clangd (authored by hokein, committed by ). Changed prior to commit: https://reviews.llvm.org/D54998?vs=175659&id=175672#toc Repository: rCTE

[PATCH] D54998: [clangd] Build and test IndexBenchmark in check-clangd

2018-11-28 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 CHANGES SINCE LAST ACTION https://reviews.llvm.org/D54998/new/ https://reviews.llvm.org/D54998 __

[PATCH] D54998: [clangd] Build and test IndexBenchmark in check-clangd

2018-11-28 Thread Haojian Wu via Phabricator via cfe-commits
hokein created this revision. hokein added a reviewer: ilya-biryukov. Herald added subscribers: kadircet, arphaman, jkorous, MaskRay, ioeric, mgorny. Include IndexBenchmark in check-clangd to make sure we won't forget to update it when doing breaking changes; also fix an out-of-date test input.