This is an archive of the discontinued LLVM Phabricator instance.

[AST] Adjust existing tests for recovery-exprs
Needs ReviewPublic

Authored by hokein on Apr 6 2020, 12:40 AM.

Details

Summary

When enabling recovery-expr (in https://reviews.llvm.org/D76696, which has been
revertedj), we need to update a few tests to adjust new diagnostic changes.

This patch updates all relevant tests, it will make the developing life
easier (if recovery-expr is enabled by default, we can revert this
patch).

Diff Detail

Event Timeline

hokein created this revision.Apr 6 2020, 12:40 AM
Herald added a project: Restricted Project. · View Herald Transcript

I'm a bit leery about this because we lose test coverage of the "live" configuration.
I know it's a pain, but maybe it's possible to keep this as a local patch/branch? :-(

sammccall resigned from this revision.Sep 18 2020, 1:41 AM

(This is obsolete now I believe, the flag has been flipped)