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

Sergio Afonso via flang-commits flang-commits at lists.llvm.org
Fri Oct 4 02:49:27 PDT 2024


================
@@ -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);
----------------
skatrak wrote:

Should this be `applyToOutermost`? I'm not so familiar with this clause, but in this patch it's only processed for the `target` construct. For example, in `target teams ompx_bare ...` this actually results in the clause being applied to both `target` and `teams`, but it would be missing lowering for the latter.

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


More information about the flang-commits mailing list