This is an archive of the discontinued LLVM Phabricator instance.

[flang] Split up synchronization, event, and error stop stmt tests
ClosedPublic

Authored by ktras on Aug 22 2022, 10:59 AM.

Details

Summary

There are some errors caught for the synchronization statements, event statements, and error stop statement that hide other errors. All of the invalid statements in the tests that do not create errors have been moved to another test file. This then allows some of those errors to be caught and shows which invalid code does or does not produce errors.

Diff Detail

Event Timeline

ktras created this revision.Aug 22 2022, 10:59 AM
Herald added a project: Restricted Project. · View Herald Transcript
Herald added a subscriber: jdoerfert. · View Herald Transcript
ktras requested review of this revision.Aug 22 2022, 10:59 AM
rouson added inline comments.Aug 24 2022, 4:30 PM
flang/test/Semantics/synchronization02b.f90
4

This suggestion also applies to other similar comments in the tests in this differential.

5

This suggestion also applies to other similar comments in the tests in this differential.

ktras updated this revision to Diff 455426.Aug 24 2022, 4:50 PM

Updated comments in some of the tests based on reviewer's feedback.

ktras marked 2 inline comments as done.Aug 24 2022, 4:51 PM
ktras updated this revision to Diff 455427.Aug 24 2022, 4:53 PM

Fix comments just updated.

rouson accepted this revision.Aug 25 2022, 11:10 AM

LGTM

This revision is now accepted and ready to land.Aug 25 2022, 11:10 AM
ktras closed this revision.EditedAug 28 2022, 10:44 AM

I pushed these changes but I forgot to include the differential revision in the commit message for the commit 21dceb3ca67c. This closes this differential.

flang/test/Semantics/synchronization03b.f90