[clang] [flang] [llvm] [mlir] [flang][OpenMP] Add frontend support for ompx_bare clause (PR #111106)
    Sergio Afonso via llvm-commits 
    llvm-commits at lists.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 llvm-commits
mailing list