[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