[Mlir-commits] [mlir] [mlir] Add the concept of ASM dialect aliases (PR #86033)

Fabian Mora llvmlistbot at llvm.org
Thu Mar 21 03:45:48 PDT 2024


fabianmcg wrote:

> On first glance this seems like a too powerful option

There's a toned down version of this idea, where only attributes and types implementing the `(Attr|Type)DialectAliasInterface` are allowed to alias.

However, that version requires the attribute or type to internally hold which dialect to use during printing (ie. not all objects can be aliased, for example `StringAttr`  wouldn't accept aliases)  and parsing requires overriding the default parsers.
The main benefit of the toned down version is it produces less overhead during parsing and printing. Nonetheless, it's cumbersome to implement.

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


More information about the Mlir-commits mailing list