[PATCH] D81452: [AST] RecoveryAST options should only be enabled for C++ only.

2020-06-09 Thread Haojian Wu via Phabricator via cfe-commits
hokein abandoned this revision. hokein added a comment. In D81452#2081838 , @sammccall wrote: > Hmm - why? > I thought the idea with the explicit `-frecovery-ast` flag was that we could > turn it on even when it's not stable. > Isn't the idea that one d

[PATCH] D81452: [AST] RecoveryAST options should only be enabled for C++ only.

2020-06-09 Thread Sam McCall via Phabricator via cfe-commits
sammccall added a comment. Hmm - why? I thought the idea with the explicit `-frecovery-ast` flag was that we could turn it on even when it's not stable. Isn't the idea that one day it will be stable for C, but not yet? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://r

[PATCH] D81452: [AST] RecoveryAST options should only be enabled for C++ only.

2020-06-09 Thread Haojian Wu via Phabricator via cfe-commits
hokein created this revision. hokein added a reviewer: sammccall. Herald added a project: clang. The flag should be off for non-c++ files even there is a "-frecovery-ast" in the commandline. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D81452 Files: clang/lib/Frontend/Compi