[all-commits] [llvm/llvm-project] 9cc08c: [clangd] Add integration test for crash handling

Sam McCall via All-commits all-commits at lists.llvm.org
Wed Oct 27 02:52:51 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 9cc08cb02fdc4f8a2b717519c3bf33b4ed3070e4
      https://github.com/llvm/llvm-project/commit/9cc08cb02fdc4f8a2b717519c3bf33b4ed3070e4
  Author: Sam McCall <sam.mccall at gmail.com>
  Date:   2021-10-27 (Wed, 27 Oct 2021)

  Changed paths:
    M clang-tools-extra/clangd/Compiler.cpp
    M clang-tools-extra/clangd/Compiler.h
    M clang-tools-extra/clangd/TUScheduler.cpp
    A clang-tools-extra/clangd/test/crash-parse.test
    A clang-tools-extra/clangd/test/crash-preamble.test
    M clang-tools-extra/clangd/tool/ClangdMain.cpp

  Log Message:
  -----------
  [clangd] Add integration test for crash handling

This replaces the test removed in 51be7061d025139ba66869d5d99c7157a3ae9edd
It is more principled and tests more critical cases: a crash while parsing.

We need two pieces of plumbing:
 - a way to re-enable the crashing #pragmas via a flag, to test parse crashes
 - a bit of reshuffling around ASTWorker execution so that we set up the
   crash handler in both sync/async modes.
   Sync mode is useful for debugging, so I tested both.

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




More information about the All-commits mailing list