[Mlir-commits] [mlir] [mlir][LLVM] Add support for constant struct with multiple fields (PR #102752)

[Tobias Gysi]

https://github.com/gysit commented:

Thanks for the updates! I have added some comments.

One thing we do not yet check are llvm.struct types nested an llvm.array. I wonder if such a verification could be added without breaking exiting users/tests. If yes, that would be nice to have. However, I believe this is currently not verified either.



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