[flang-commits] [clang] [flang] [llvm] [mlir] [flang][OpenMP] Add frontend support for ompx_bare clause (PR #111106)

[Ivan R. Ivanov via flang-commits]

================
@@ -1103,6 +1105,13 @@ bool ConstructDecompositionT<C, H>::applyClause(
   return applyToOutermost(node);
 }
 
+template <typename C, typename H>
+bool ConstructDecompositionT<C, H>::applyClause(
+    const tomp::clause::OmpxBareT<TypeTy, IdTy, ExprTy> &clause,
+    const ClauseTy *node) {
+  return applyToAll(node);
----------------
ivanradanov wrote:

We need to know whether ompx_bare was present, because the runtime _must_ run the target region with the number of threads and teams specified (instead of `thread_limit` and `num_teams` being just mere hints otherwise). So I think the easiest way to do that is to represent the clause in the dialect. I am fine with having in on either `omp.target` or `omp.teams` (or both) and which one we have it on only has minor implications on the op verification and translation implementation and does not matter much in my opinion, so having it on the `omp.target` should be fine in my opinion?

https://github.com/llvm/llvm-project/pull/111106