[Mlir-commits] [mlir] [MLIR][Vector] Add support for inner-parallel masked multi-reductions (PR #126722)

Kunwar Grover llvmlistbot at llvm.org
Tue Feb 11 06:36:36 PST 2025


================
@@ -310,9 +310,16 @@ struct TwoDimMultiReductionToElementWise
                                 PatternRewriter &rewriter) const override {
     auto maskableOp =
         cast<vector::MaskableOpInterface>(multiReductionOp.getOperation());
-    if (maskableOp.isMasked())
-      // TODO: Support masking.
-      return failure();
+
+    Operation *rootOp;
+    Value mask = nullptr;
+    if (maskableOp.isMasked()) {
+      rewriter.setInsertionPoint(maskableOp.getMaskingOp());
----------------
Groverkss wrote:

Can we move this lower to when we start creating operations and just add rewriter.setInsertionPoint(rootOp) ?

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


More information about the Mlir-commits mailing list