[clang] [lldb] [Clang][SPIRV] Add __spirv_event_t builtin type (PR #207077)
via cfe-commits
cfe-commits at lists.llvm.org
Fri Jul 17 09:49:55 PDT 2026
elizabethandrews wrote:
> We should validate that it can round trip serialization correctly,
Can you clarify what you mean by this. I test serialization in `AST/spirv-event-type.cpp`, similar to what is done here - https://github.com/llvm/llvm-project/pull/94830
> test mangling behavior (oops, probably need to update the demangler as well), etc.
Mangling has been tested in `CodeGenCXX/spirv-event-type.cpp`. I am not sure how to write a test for demangling. I could not find tests for opaque builtins I based this patch on. FWIW I tested locally and it works. We don't need to update the demangler since we use u<len><name> mangling and this is already handled by the demangler.
```
$ ./build/bin/llvm-cxxfilt _Z12consumeEventu15__spirv_event_t
consumeEvent(__spirv_event_t)
$ ./build/bin/llvm-cxxfilt _Z4funcIu15__spirv_event_tEvT_
void func<__spirv_event_t>(__spirv_event_t)
```
https://github.com/llvm/llvm-project/pull/207077
More information about the cfe-commits
mailing list