[Mlir-commits] [mlir] [mlir][vector] Always print the in_bounds attribute (PR #96031)

Benjamin Maxwell llvmlistbot at llvm.org
Fri Jun 21 09:31:34 PDT 2024


https://github.com/MacDue approved this pull request.

I think always displaying it is fine :+1: -- though I've never really understood the utility of the `in_bounds` attribute. It's mostly just passed around (and not really used). There's `MaterializeTransferMask` which simply replaces out-of-bounds transfer_reads/writes with in-bound ones with a mask, which makes it seem like it mostly duplicates the functionality of a mask, so I question if it's worth the extra complexity.   

(removing the attribute is obviously out of scope for this PR though)

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


More information about the Mlir-commits mailing list