================
@@ -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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to