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

Katherine Rasmussen via Phabricator via flang-commits flang-commits at lists.llvm.org
Fri May 27 15:10:49 PDT 2022


ktras marked an inline comment as done.
ktras added inline comments.


================
Comment at: flang/lib/Evaluate/intrinsics.cpp:1044
+    {"co_sum",
+        {{"a", AnyNumeric, Rank::elemental, Optionality::required,
+             common::Intent::InOut},
----------------
klausler wrote:
> "A=", "STAT=", and "ERRMSG=" are not allowed to be coindexed objects (16.9.50); how will you enforce and test this restriction?
Thanks for the feedback and sorry that it took me so long to respond.
I have now added a check for coindexed objects for these arguments.


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

https://reviews.llvm.org/D114134



More information about the flang-commits mailing list