[all-commits] [llvm/llvm-project] e6be5c: [clangd] Remove the recovery-ast options.

Sam McCall via All-commits all-commits at lists.llvm.org
Wed Jan 20 02:36:33 PST 2021


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: e6be5c7cd6d227144f874623e2764890f80cad32
      https://github.com/llvm/llvm-project/commit/e6be5c7cd6d227144f874623e2764890f80cad32
  Author: Sam McCall <sam.mccall at gmail.com>
  Date:   2021-01-20 (Wed, 20 Jan 2021)

  Changed paths:
    M clang-tools-extra/clangd/ClangdServer.cpp
    M clang-tools-extra/clangd/ClangdServer.h
    M clang-tools-extra/clangd/Compiler.cpp
    M clang-tools-extra/clangd/Compiler.h
    M clang-tools-extra/clangd/tool/ClangdMain.cpp

  Log Message:
  -----------
  [clangd] Remove the recovery-ast options.

These force a couple of flags or that are now on by default.
So the flags don't currently do anything unless the compile command has
-fno-recovery-ast explicitly.

(For turning recovery *off* for debugging we can inject the flag with config)

This leaves the command-line flags around with no effect, I'm planning to add
a "retired flag" mechanism shortly in a separate patch.

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




More information about the All-commits mailing list