[flang-commits] [PATCH] D135840: [flang] Add a semantics test for atomic_ref

[Damian Rouson via Phabricator via flang-commits]

rouson accepted this revision.
rouson added inline comments.
This revision is now accepted and ready to land.


================
Comment at: flang/test/Semantics/atomic10.f90:39
+  call atomic_ref(val_logical, atom_logical[1], status)
+  call atomic_ref(val_logical, atom_logical, stat=status)
+  call atomic_ref(value=val_logical, atom=atom_logical, stat=status)
----------------
Please add a call with atom and stat as keyword arguments but not value.


================
Comment at: flang/test/Semantics/atomic10.f90:42
+  call atomic_ref(value=val_logical, atom=atom_logical)
+  call atomic_ref(value=val_logical, atom=atom_logical, stat=status)
+  call atomic_ref(stat=status, atom=atom_logical, value=val_logical)
----------------
I think 40 and 42 are identical.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D135840/new/

https://reviews.llvm.org/D135840