This is an archive of the discontinued LLVM Phabricator instance.

[clangd] Turn on RecoveryAST for clangd by default.
ClosedPublic

Authored by hokein on Jun 12 2020, 7:09 AM.

Diff Detail

Event Timeline

hokein created this revision.Jun 12 2020, 7:09 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 12 2020, 7:09 AM
sammccall accepted this revision.Jun 12 2020, 11:04 AM
This revision is now accepted and ready to land.Jun 12 2020, 11:04 AM
sammccall added inline comments.Jun 12 2020, 11:05 AM
clang-tools-extra/clangd/tool/ClangdMain.cpp
285

Ah actually in other options, we set the default in the struct and reference it here. Can we do that here too?

hokein updated this revision to Diff 270682.Jun 15 2020, 1:38 AM
hokein marked an inline comment as done.

address review comment.

This revision was automatically updated to reflect the committed changes.