[Mlir-commits] [mlir] [mlir][vector] Update the internal representation of in_bounds (PR #100336)

Benjamin Maxwell llvmlistbot at llvm.org
Tue Aug 13 09:49:13 PDT 2024


================
@@ -1720,13 +1720,13 @@ def Vector_TransferWriteOp :
                    "ValueRange":$indices,
                    "AffineMapAttr":$permutationMapAttr,
                    "Value":$mask,
-                   "ArrayAttr":$inBoundsAttr)>,
+                   "DenseBoolArrayAttr":$inBoundsAttr)>,
     /// 2. Builder with type inference that sets an empty mask (variant with attrs).
     OpBuilder<(ins "Value":$vector,
                    "Value":$dest,
                    "ValueRange":$indices,
                    "AffineMapAttr":$permutationMapAttr,
-                   "ArrayAttr":$inBoundsAttr)>,
----------------
MacDue wrote:

I think it'd make things simpler if these builds just took an `ArrayRef<bool>` rather than a `DenseBoolArrayAttr`. 

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


More information about the Mlir-commits mailing list