================ @@ -230,8 +230,15 @@ bool DataSharingProcessor::needBarrier() { return false; } -void DataSharingProcessor::insertBarrier() { - if (needBarrier()) +void DataSharingProcessor::insertBarrier( + mlir::omp::PrivateClauseOps *clauseOps) { ---------------- luporl wrote:
```suggestion mlir::omp::PrivateClauseOps &clauseOps) { ``` If `clauseOps` can't be NULL, it would be better to use a reference. https://github.com/llvm/llvm-project/pull/140092 _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits