This is an archive of the discontinued LLVM Phabricator instance.

[flang] Write semantics test for atomic_fetch_or
ClosedPublic

Authored by ktras on Aug 23 2022, 5:54 PM.

Details

Summary

Write a semantics test for the atomic intrinsic subroutine, atomic_fetch_or.

Diff Detail

Event Timeline

ktras created this revision.Aug 23 2022, 5:54 PM
Herald added a project: Restricted Project. · View Herald Transcript
Herald added a subscriber: jdoerfert. · View Herald Transcript
ktras requested review of this revision.Aug 23 2022, 5:54 PM
rouson added inline comments.Aug 30 2022, 1:18 PM
flang/test/Semantics/atomic7.f90
21 ↗(On Diff #455034)

There's probably not a specific reason to test any stat argument kind other than the default integer kind.

70 ↗(On Diff #455034)

coindexed_status is not coindexed

127 ↗(On Diff #455034)

This line appears to be standard-conforming.

ktras updated this revision to Diff 456773.Aug 30 2022, 1:24 PM

Make changes in test based on feedback.

ktras marked 3 inline comments as done.Aug 30 2022, 1:25 PM
rouson accepted this revision.Aug 30 2022, 1:31 PM

LGTM

This revision is now accepted and ready to land.Aug 30 2022, 1:31 PM
This revision was automatically updated to reflect the committed changes.