-
Notifications
You must be signed in to change notification settings - Fork 148
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add spirv_reflect.hpp #152
Comments
Can you elaborate? I'm not sure I'm following. |
Cpp wrapper around SPIRV-Reflect C headers. Just like SPIRV-Headers got spirv.hpp and Vulkan SDK got vulkan.hpp, it would be nice to have spirv-reflect.hpp. |
Just so I understand the ask correctly. This would be a purely cosmetic addition since the spirv_reflect.h already has C++ code in with if the compiler is C++. So essentially spirv_reflect.hpp would just be a file that includes spirv_reflect.h? Is my understanding correct? |
Not exactly. I suggest same thing, that was done with vulkan-sdk here: CPP stl wrapper around C header of spirv-reflect for improving experience working within cpp. Since Vulkan-Hpp seems to be autogenerated, there might be a way to integrating same solution to generate hpp headers for spirv-reflect. |
SPIRV-Headers already contain spriv.hpp, please consider adding spriv_reflect.hpp
The text was updated successfully, but these errors were encountered: