[Mlir-commits] [mlir] [mlir][python] Add bindings for mlirDenseElementsAttrGet (PR #91389)

Stella Laurenzo llvmlistbot at llvm.org
Tue May 21 13:11:25 PDT 2024


stellaraccident wrote:

> That makes sense. It sounds like `DenseArrayAttr` would be more appropriate for what I'm trying to do. If the idea is to move away from `DenseElementsAttr`, then maybe it's better not to expose new APIs for it?

It exists and is still used. We can certainly add ergonomics like this to it if helpful. I just haven't had time to review the patch in detail with an eye to landing. 

But yeah, most mature implementations have been limiting their use of that attribute generally for a long time, due to its design flaws.

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


More information about the Mlir-commits mailing list