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

Document required descriptor set annotations #64

Closed
thatcosmonaut opened this issue Jun 27, 2024 · 1 comment
Closed

Document required descriptor set annotations #64

thatcosmonaut opened this issue Jun 27, 2024 · 1 comment

Comments

@thatcosmonaut
Copy link
Owner

Shader resources and uniform buffers have to be declared using a particular descriptor set order in the shader source to be usable with Vulkan. (Is this also true for Metal?) We should document this in the header so that people don't have to look at the examples to find them.

@TheSpydog
Copy link
Collaborator

Can confirm that Metal definitely needs some special documentation for expected resource order. Uniforms, readonly storage buffers, read-write storage buffers, and vertex buffers all share the same "buffer" address space and are expected to be in a particular order. Similarly, readonly/read-write storage textures and regular textures share a mutual "texture" address space.

I'll write up the Metal docs once the Vulkan docs have been added in, since I imagine they'll share the same comment block.

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

No branches or pull requests

3 participants