================ @@ -97,11 +97,13 @@ class OpenACCClauseCIREmitter final // Handle a clause affected by the 'device-type' to the point that they need // to have the attributes added in the correct/corresponding order, such as - // 'num_workers' or 'vector_length' on a compute construct. - mlir::ArrayAttr - handleDeviceTypeAffectedClause(mlir::ArrayAttr existingDeviceTypes, - mlir::Value argument, - mlir::MutableOperandRange &argCollection) { + // 'num_workers' or 'vector_length' on a compute construct. For cases where we + // don't have an argument that needs to be added to an additional one (such as ---------------- andykaylor wrote:
The meaning of "we don't have an argument that needs to be added to an additional one" is unclear. Does this mean, "the clause doesn't have an argument"? https://github.com/llvm/llvm-project/pull/136626 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits