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

Sergio Afonso llvmlistbot at llvm.org
Wed Oct 23 07:53:46 PDT 2024


================
@@ -1672,8 +1672,9 @@ void TargetOp::build(OpBuilder &builder, OperationState &state,
   // TODO Store clauses in op: allocateVars, allocatorVars, inReductionVars,
   // inReductionByref, inReductionSyms.
   TargetOp::build(builder, state, /*allocate_vars=*/{}, /*allocator_vars=*/{},
-                  makeArrayAttr(ctx, clauses.dependKinds), clauses.dependVars,
-                  clauses.device, clauses.hasDeviceAddrVars, clauses.ifExpr,
+                  clauses.bare, makeArrayAttr(ctx, clauses.dependKinds),
+                  clauses.dependVars, clauses.device, clauses.hasDeviceAddrVars,
+                  clauses.ifExpr,
                   /*in_reduction_vars=*/{}, /*in_reduction_byref=*/nullptr,
                   /*in_reduction_syms=*/nullptr, clauses.isDevicePtrVars,
                   clauses.mapVars, clauses.nowait, clauses.privateVars,
----------------
skatrak wrote:

Nit: Looks like a formatting artifact.
```suggestion
                  clauses.ifExpr, /*in_reduction_vars=*/{},
                  /*in_reduction_byref=*/nullptr, /*in_reduction_syms=*/nullptr,
                  clauses.isDevicePtrVars, clauses.mapVars, clauses.nowait,
                  clauses.privateVars,
```

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


More information about the Mlir-commits mailing list