[clang] [lldb] [Clang][SPIRV] Add __spirv_event_t builtin type (PR #207077)
Aaron Ballman via cfe-commits
cfe-commits at lists.llvm.org
Thu Jul 16 09:41:54 PDT 2026
https://github.com/AaronBallman commented:
I think the test coverage is pretty light for adding a new builtin type. We should validate that it can round trip serialization correctly, add coverage for C as well, test mangling behavior (oops, probably need to update the demangler as well), etc.
I'm not familiar enough with the SPIR-V spec to know what kinds of other things need to be handled though. I added @Keenuts and @VyacheslavLevytskyy as reviewers because they're backend SPIR-V folks, so they may have more ideas on the kinds of restrictions and corner cases we need to be considering in the frontend.
https://github.com/llvm/llvm-project/pull/207077
More information about the cfe-commits
mailing list