[Mlir-commits] [mlir] [mlir][spirv] Fix int type declaration duplication when serializing (PR #143108)

[Davide Grohmann]

davidegrohmann wrote:

> I thought we have some tests that use spir-v when present:
> 
> https://github.com/llvm/llvm-project/blob/e4060d3beab3b525b49baaa15484e3c595740a2f/llvm/test/CodeGen/SPIRV/empty.ll#L2
> 
> Could we do something similar in mlir spirv serialization?
> 
> If this is not straightforward, validating by hand is also fine IMO.

Thanks for the suggestion I have added that to the test. I have also validated manually the SPIR-V module produced by that test and it passes locally for me.

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