[Mlir-commits] [mlir] [mlir] Add loaded URI attribute type (PR #67276)

Jeff Niu llvmlistbot at llvm.org
Wed Sep 27 13:13:16 PDT 2023


Mogball wrote:

> Even today when we landed ArrayAttr, it could have been anywhere

I think that ArrayAttr makes sense to be "builtin", but today, I would actually argue against even having an ArrayAttr. These days, uses of ArrayAttr tend to be tech debt (using an ArrayAttr of size 2 to represent LLVM complex constants, for example, is insanity).

I think having a URI dialect makes sense, because then extensions can be registered and managed through the dialect itself, rather than building yet another extension system.

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


More information about the Mlir-commits mailing list