[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