[PATCH] D151972: [flang][openacc] Add parsing support for dim in gang clause

Anastasia Stulova via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jun 4 13:02:36 PDT 2023


Anastasia added inline comments.


================
Comment at: flang/test/Parser/acc-unparse.f90:57
+
+  !$acc loop gang(num: gangNum, dim: gangDim)
+  do i = 1, 10
----------------
Btw `dim` and `num` should not appear together, s2.9.2  paragraph 1 says:

```
The dim argument must be a constant positive integer with value 1, 2, or 3. <...> The number of gangs in dimension d is controlled by the parallel construct; the num argument is not allowed.
```

There are a number of other restrictions, for example the value of `dim` can only be  an integer literal in the range 1-3 and  it has to appear only once. But maybe you plan to add the diagnostics later?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D151972



More information about the llvm-commits mailing list