[Frontend][OpenMP] Add order clause as allowed for distribute

[kparzysz]

Starting from OpenMP spec 5.1, the order clause is allowed on the distribute directive.

[llvmbot]

@llvm/pr-subscribers-flang-openmp

Author: Krzysztof Parzyszek (kparzysz)

Changes

Starting from OpenMP spec 5.1, the order clause is allowed on the distribute directive.

---

Full diff: https://github.com/llvm/llvm-project/pull/90689.diff

1 Files Affected:

• (modified) llvm/include/llvm/Frontend/OpenMP/OMP.td (+2-1)

diff --git a/llvm/include/llvm/Frontend/OpenMP/OMP.td b/llvm/include/llvm/Frontend/OpenMP/OMP.td
index e91169e8da1aa5..9c9bfb4c59fc77 100644
--- a/llvm/include/llvm/Frontend/OpenMP/OMP.td
+++ b/llvm/include/llvm/Frontend/OpenMP/OMP.td
@@ -1173,7 +1173,8 @@ def OMP_Distribute : Directive<"distribute"> {
     VersionedClause<OMPC_Private>,
     VersionedClause<OMPC_FirstPrivate>,
     VersionedClause<OMPC_LastPrivate>,
-    VersionedClause<OMPC_Allocate>
+    VersionedClause<OMPC_Allocate>,
+    VersionedClause<OMPC_Order, 51>
   ];
   let allowedOnceClauses = [
     VersionedClause<OMPC_Collapse>,

[alexey-bataev]

It requires testing and proper codegen

[kiranchandramohan]

For Flang this is only currently used for checking which clauses are allowed in which constructs. At the moment this will lead to an error. With this change it will pass. Adding a test might be good.

$ ./bin/flang-new -fopenmp nb.f90
error: Semantic errors in nb.f90
./nb.f90:10:20: error: ORDER clause is not allowed on the DISTRIBUTE directive
    !$omp distribute order(concurrent)

Please wait for @alexey-bataev for the Clang side.

[sandeepkosuri]

I am working on codegen support of order clause modifiers and I would like this PR to be checked in. Any updates on this @kparzysz?