[PATCH] D70684: [clangd] Shutdown cleanly on signals.

2019-11-27 Thread Sam McCall via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG19ac0eaf07e6: [clangd] Shutdown cleanly on signals. (authored by sammccall). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70684/new/ https://reviews.llvm.o

[PATCH] D70684: [clangd] Shutdown cleanly on signals.

2019-11-26 Thread pre-merge checks [bot] via Phabricator via cfe-commits
merge_guards_bot added a comment. Build result: pass - 60240 tests passed, 0 failed and 732 were skipped. Log files: console-log.txt , CMakeCache.txt

[PATCH] D70684: [clangd] Shutdown cleanly on signals.

2019-11-26 Thread Sam McCall via Phabricator via cfe-commits
sammccall updated this revision to Diff 231014. sammccall added a comment. Don't close the input stream early. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70684/new/ https://reviews.llvm.org/D70684 Files: clang-tools-extra/clangd/CMakeLists.tx

[PATCH] D70684: [clangd] Shutdown cleanly on signals.

2019-11-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. Thanks, LGTM! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70684/new/ https://reviews.llvm.org/D70684 __

[PATCH] D70684: [clangd] Shutdown cleanly on signals.

2019-11-26 Thread pre-merge checks [bot] via Phabricator via cfe-commits
merge_guards_bot added a comment. Build result: fail - 60239 tests passed, 1 failed and 732 were skipped. failed: Clangd.Clangd/exit-signal.test Log files: console-log.txt , CMakeCache.txt

[PATCH] D70684: [clangd] Shutdown cleanly on signals.

2019-11-26 Thread Sam McCall via Phabricator via cfe-commits
sammccall updated this revision to Diff 231012. sammccall added a comment. Don't require temp file to already exist! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70684/new/ https://reviews.llvm.org/D70684 Files: clang-tools-extra/clangd/CMakeLi

[PATCH] D70684: [clangd] Shutdown cleanly on signals.

2019-11-25 Thread pre-merge checks [bot] via Phabricator via cfe-commits
merge_guards_bot added a comment. Herald added a subscriber: dexonsmith. Build result: fail - 60239 tests passed, 1 failed and 732 were skipped. failed: Clangd.Clangd/exit-signal.test Log files: console-log.txt

[PATCH] D70684: [clangd] Shutdown cleanly on signals.

2019-11-25 Thread Sam McCall via Phabricator via cfe-commits
sammccall created this revision. sammccall added a reviewer: kadircet. Herald added subscribers: cfe-commits, usaxena95, jfb, arphaman, jkorous, MaskRay, ilya-biryukov, mgorny. Herald added a reviewer: jfb. Herald added a project: clang. This avoids leaking PCH files if editors don't use the LSP