[Mlir-commits] [mlir] [mlir][gpu] Eliminate redundant gpu.barrier ops (PR #71575)

[Oleksandr Alex Zinenko]

================
@@ -1139,6 +1139,29 @@ void ShuffleOp::build(OpBuilder &builder, OperationState &result, Value value,
         mode);
 }
 
+//===----------------------------------------------------------------------===//
+// BarrierOp
+//===----------------------------------------------------------------------===//
+
+namespace {
+
+/// Remove gpu.barrier after gpu.barrier, the threads are already synchronized!
+LogicalResult foldRedundantGpuBarrierOps(BarrierOp op,
----------------
ftynse wrote:

Naming nit: "fold" suggests this is a _folder_ rather than a canonicalization _pattern_. Please consider a different name.

https://github.com/llvm/llvm-project/pull/71575