Skip to content
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

Open
GloryOfNight opened this issue Jul 6, 2022 · 4 comments
Open

Add spirv_reflect.hpp #152

GloryOfNight opened this issue Jul 6, 2022 · 4 comments

Comments

@GloryOfNight
Copy link

SPIRV-Headers already contain spriv.hpp, please consider adding spriv_reflect.hpp

@chaoticbob
Copy link
Contributor

Can you elaborate? I'm not sure I'm following.

@GloryOfNight
Copy link
Author

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.

@chaoticbob
Copy link
Contributor

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?

@GloryOfNight
Copy link
Author

Not exactly.

I suggest same thing, that was done with vulkan-sdk here:
https://www.khronos.org/assets/uploads/developers/library/2016-vulkan-devu-seoul/4-Vulkan-HPP.pdf
and
https://github.com/KhronosGroup/Vulkan-Hpp

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants