[PATCH] D126721: [flang] Add lcobound to the list of intrinsics and update test

Katherine Rasmussen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 3 14:04:08 PDT 2022


ktras planned changes to this revision.
ktras added inline comments.


================
Comment at: flang/test/Semantics/lcobound.f90:10
+  real real_coarray[*]
+  logical non_integer, logical_coarray[*]
   integer, allocatable :: lcobounds(:)
----------------
craig.rasmussen wrote:
> How about testing for corank>1?
I have added some coarrays with a corank>1 to pass to the `coarray` argument now.  In doing so, I discovered that the restriction I tried to add to the `dim` argument, requiring it to be within the range of the `corank` doesn't work when the `corank` is above 1. So I am going back to work on fixing that before I update the patch.

Thanks for the suggestion!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126721



More information about the llvm-commits mailing list