[flang-commits] [PATCH] D116133: [flang] Expand coverage of co_sum unit test

[Katherine Rasmussen via Phabricator via flang-commits]

ktras requested changes to this revision.
ktras added a comment.
This revision now requires changes to proceed.

Please update some of the expected error messages as I suggest in the inline comments. Please also replace the `‘` character with `'` where they exist in the expected error messages.



================
Comment at: flang/test/Semantics/collectives01.f90:64
+  
+  ! 'result_image' argument shall be a scalar
+  !ERROR: too many actual arguments for intrinsic 'co_sum'
----------------



================
Comment at: flang/test/Semantics/collectives01.f90:65
+  ! 'result_image' argument shall be a scalar
+  !ERROR: too many actual arguments for intrinsic 'co_sum'
+  call co_sum(i, result_image=bool)
----------------



================
Comment at: flang/test/Semantics/collectives01.f90:69
+  ! 'result_image' argument shall be an integer scalar
+  !ERROR: too many actual arguments for intrinsic 'co_sum'
+  call co_sum(c, result_image=integer_array)
----------------



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116133