[PATCH] D81163: [AST][RecoveryExpr] Preserve the AST for invalid conditions.

2020-07-20 Thread Haojian Wu via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG684e416ef136: [AST][RecoveryExpr] Preserve the AST for invalid conditions. (authored by hokein). Changed prior to commit: https://reviews.llvm.org/D81163?vs=270413&id=279213#toc Repository: rG LLVM G

[PATCH] D81163: [AST][RecoveryExpr] Preserve the AST for invalid conditions.

2020-06-12 Thread Haojian Wu via Phabricator via cfe-commits
hokein updated this revision to Diff 270413. hokein added a comment. rebase and adjust the test. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D81163/new/ https://reviews.llvm.org/D81163 Files: clang/lib/Sema/SemaExpr.cpp clang/test/AST/ast-dum

[PATCH] D81163: [AST][RecoveryExpr] Preserve the AST for invalid conditions.

2020-06-05 Thread Sam McCall via Phabricator via cfe-commits
sammccall accepted this revision. sammccall added a comment. This revision is now accepted and ready to land. Nice! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D81163/new/ https://reviews.llvm.org/D81163

[PATCH] D81163: [AST][RecoveryExpr] Preserve the AST for invalid conditions.

2020-06-04 Thread Haojian Wu via Phabricator via cfe-commits
hokein created this revision. hokein added a reviewer: sammccall. Herald added a project: clang. Tested with enabling "-frecovery-ast", only 1 failure, which is an improvement of secondary diagnostic. -- error: 'error' diagnostics seen but not expected: File /workspace/llvm-project/clang/