[Mlir-commits] [mlir] [mlir][llvm] adds an attribute for the module level assembly (PR #151318)

[Tobias Gysi]

gysit wrote:

@gitoleg 

Looking at the language ref and some code examples in LLVM it looks like there can be many of these module level assembly blocks? Wouldn't it make more sense to implement them as a module level operation similar to GlobalOp / AliasOp etc. I mean an operation that needs to be nested directly within the moduleOp. 

Or do I miss something and there is just one inline assembly string per module? Then an attribute makes more sense.

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