[PATCH] D81739: [clangd] Turn on RecoveryAST for clangd by default.

2020-06-15 Thread Haojian Wu via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG1e87261ba17c: [clangd] Turn on RecoveryAST for clangd by default. (authored by hokein). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D81739/new/ https://rev

[PATCH] D81739: [clangd] Turn on RecoveryAST for clangd by default.

2020-06-15 Thread Haojian Wu via Phabricator via cfe-commits
hokein updated this revision to Diff 270682. hokein marked an inline comment as done. hokein added a comment. address review comment. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D81739/new/ https://reviews.llvm.org/D81739 Files: clang-tools-ext

[PATCH] D81739: [clangd] Turn on RecoveryAST for clangd by default.

2020-06-12 Thread Sam McCall via Phabricator via cfe-commits
sammccall added inline comments. Comment at: clang-tools-extra/clangd/tool/ClangdMain.cpp:285 +desc("Preserve expressions in AST for broken code (C++ only)."), +init(true), }; Ah actually in other options, we set the default in the struct and reference i

[PATCH] D81739: [clangd] Turn on RecoveryAST for clangd by default.

2020-06-12 Thread Haojian Wu via Phabricator via cfe-commits
hokein created this revision. hokein added a reviewer: sammccall. Herald added subscribers: usaxena95, kadircet, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D81739 Files: clang-tools-extra/clangd/tool