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

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


================
@@ -132,7 +131,7 @@ struct TransferReadPermutationLowering
     }
 
     // Transpose in_bounds attribute.
-    ArrayAttr newInBoundsAttr =
+    DenseBoolArrayAttr newInBoundsAttr =
         inverseTransposeInBoundsAttr(rewriter, op.getInBounds(), permutation);
----------------
MacDue wrote:

Also here, I think `inverseTransposeInBoundsAttr` can be removed, and instead we can now just use the existing `applyPermutation`:  

```
SmallVector<bool> newInBounds = applyPermutation(op.getInBounds(), permutation);
```

(and just delay constructing the `DenseBoolArrayAttr`)

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


More information about the Mlir-commits mailing list