================
@@ -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

Reply via email to