[Mlir-commits] [mlir] [mlir][spirv] Truncate Literal String size at max number words (PR #142916)

Davide Grohmann llvmlistbot at llvm.org
Thu Jun 5 04:53:28 PDT 2025


davidegrohmann wrote:

> Thank you for your contribution!
> 
> If possible, could you please try to add a test to exercise this change? I believe a target test should do the trick (see `mlir/test/Target/SPIRV`). Those tests serialize given MLIR into SPIR-V and then deserialize the SPIR-V back to MLIR. So, I would hope you could see a truncation after the roundtrip.

It has been tricky to add a good test for this. Something needs to trigger the encoding of a string literal with more than 262140 chars.
Any suggestion on which spirv operation to use in such a test?

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


More information about the Mlir-commits mailing list