[all-commits] [llvm/llvm-project] 19ac0e: [clangd] Shutdown cleanly on signals.

Sam McCall via All-commits all-commits at lists.llvm.org
Wed Nov 27 03:45:16 PST 2019


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 19ac0eaf07e60173baa7ee77fa11568c30b87455
      https://github.com/llvm/llvm-project/commit/19ac0eaf07e60173baa7ee77fa11568c30b87455
  Author: Sam McCall <sam.mccall at gmail.com>
  Date:   2019-11-27 (Wed, 27 Nov 2019)

  Changed paths:
    M clang-tools-extra/clangd/CMakeLists.txt
    M clang-tools-extra/clangd/JSONTransport.cpp
    A clang-tools-extra/clangd/Shutdown.cpp
    A clang-tools-extra/clangd/Shutdown.h
    A clang-tools-extra/clangd/test/exit-eof.test
    A clang-tools-extra/clangd/test/exit-signal.test
    M clang-tools-extra/clangd/tool/ClangdMain.cpp

  Log Message:
  -----------
  [clangd] Shutdown cleanly on signals.

Summary:
This avoids leaking PCH files if editors don't use the LSP shutdown protocol.

This is one fix for https://github.com/clangd/clangd/issues/209
(Though I think we should *also* be unlinking the files)

Reviewers: kadircet, jfb

Subscribers: mgorny, ilya-biryukov, MaskRay, jkorous, arphaman, jfb, usaxena95, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D70684




More information about the All-commits mailing list