[Mlir-commits] [mlir] [mlir][spirv] Add spirv-to-llvm conversion for group operations (PR #115501)

Victor Perez llvmlistbot at llvm.org
Fri Nov 8 08:17:36 PST 2024


================
@@ -1722,7 +1909,50 @@ void mlir::populateSPIRVToLLVMConversionPatterns(
       ReturnPattern, ReturnValuePattern,
 
       // Barrier ops
-      ControlBarrierPattern>(patterns.getContext(), typeConverter);
+      ControlBarrierPattern,
+
+      // Group reduction operations
+      GroupReducePattern<spirv::GroupIAddOp>,
+      GroupReducePattern<spirv::GroupFAddOp>,
+      GroupReducePattern<spirv::GroupFMinOp>,
+      GroupReducePattern<spirv::GroupUMinOp>,
+      GroupReducePattern<spirv::GroupSMinOp, /*Signed*/ true>,
----------------
victor-eds wrote:

```suggestion
      GroupReducePattern<spirv::GroupSMinOp, /*Signed=*/true>,
```

`clang-format` likes this better (same for below)

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


More information about the Mlir-commits mailing list