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

Andrzej Warzyński llvmlistbot at llvm.org
Fri Aug 16 10:00:11 PDT 2024


banach-space wrote:

@joker-eph IIUC, you feel that this format (consistent with the in-memory representation) would be more intuitive:

```mlir
vector.transfer_read %arg0[%0, %1], %cst {in_bounds = array<i1: true>, permutation_map = #map3} : memref<12x16xf32>, vector<8xf32>
```

Which brings us back to @dcaballe 's comment:

> I think it would be important to minimize the conflicts so if we could keep the in_bounds = [true, true] textual form that would be amazing.

Lets wait for Diego to get back to this (and all the new context since his original comment😅). Thanks for all the discussion so far 🙏🏻  

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


More information about the Mlir-commits mailing list