[flang-commits] [PATCH] D114181: Add semantics unit test for SYNC ALL statement

[Katherine Rasmussen via Phabricator via flang-commits]

ktras added inline comments.


================
Comment at: flang/test/Semantics/synchronization01.f90:23
+  !ERROR: expected execution part construct
+  sync all(invalid_stat_specifier=sync_status)
+
----------------
An addition of a `sync all statement` with the correct type for the `errmsg` specifier, but the incorrect name of the specifier, as you have done here for `sync_status`, would be a good addition. Perhaps the incorrect specifier name could be close to the correct one, such as saying `errormsg=error_message` instead of the correct `errmsg=error_message`.


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

https://reviews.llvm.org/D114181