This is an archive of the discontinued LLVM Phabricator instance.

[clang] Fix ForStmt mustprogress handling
ClosedPublic

Authored by atmnpatel on Nov 9 2020, 6:29 AM.

Details

Summary

D86841 had an error where for statements with no conditional were
required to make progress. This is not true, this patch removes that
line, and adds regression tests.

Diff Detail

Event Timeline

atmnpatel created this revision.Nov 9 2020, 6:29 AM
Herald added a project: Restricted Project. · View Herald TranscriptNov 9 2020, 6:29 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
atmnpatel requested review of this revision.Nov 9 2020, 6:29 AM
This revision is now accepted and ready to land.Nov 9 2020, 6:57 AM
This revision was landed with ongoing or failed builds.Nov 9 2020, 8:38 AM
This revision was automatically updated to reflect the committed changes.