[PATCH] D114134: [flang] Add co_sum to the list of intrinsics and update test

Katherine Rasmussen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 27 15:08:17 PDT 2022


ktras updated this revision to Diff 432641.
ktras retitled this revision from "[flang] Add evaluation for the collective intrinsic, co_sum" to "[flang] Add co_sum to the list of intrinsics and update test".
ktras edited the summary of this revision.
ktras added reviewers: rouson, PeteSteinfeld, jeanPerier.
ktras added a comment.
Herald added a project: All.

- Fix rank of the `co_sum` dummy argument `a` in `flang/lib/Evaluate/intrinsics.cpp`
- Add check for and error if a coindexed object is passed to dummy arguments `a`, `stat`, and `errmsg` in `co_sum`, and dummy arguments `from`, `to`, `stat`, and `errmsg` in `move_alloc`.
- Add semantics test to ensure that coindexed error is being caught in calls to `move_alloc`.
- Update semantics test for `co_sum` by fixing ERROR directives for the new error and adding one non-standard conforming call.


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

https://reviews.llvm.org/D114134

Files:
  flang/lib/Evaluate/intrinsics.cpp
  flang/test/Semantics/collectives01.f90
  flang/test/Semantics/move_alloc.f90

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D114134.432641.patch
Type: text/x-patch
Size: 5689 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220527/131ddc74/attachment.bin>


More information about the llvm-commits mailing list