-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Add mesh shading info to naga IR #8104
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
base: trunk
Are you sure you want to change the base?
Add mesh shading info to naga IR #8104
Conversation
I'm unsure why the MSRV minimal versions thing is failing, it doesn't look related to this PR since it happens in another crate and this PR doesn't touch anything cargo. |
Should be fixed with #8112, thanks @andyleiserson! |
@@ -966,6 +984,7 @@ pub enum Binding { | |||
/// Optional `blend_src` index used for dual source blending. | |||
/// See <https://www.w3.org/TR/WGSL/#attribute-blend_src> | |||
blend_src: Option<u32>, | |||
per_primitive: bool, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we have some docs for this? The Location
variant's docs talk about passing values from the vertex stage to the fragment stage; we should make sure the story told here makes sense for readers working on mesh shaders too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've tried to write some docs. Let me know if you have further comments.
Connections
Mostly split off from #7390
Works towards #7197
Description
This PR adds mesh shading info to naga IR so that parsers and writers have an interface to use.
Testing
No testing yet (coming in later PRs, the code here has been tested in #7390)
Squash or Rebase?
Squash
Checklist
cargo fmt
.taplo format
.cargo clippy --tests
. If applicable, add:--target wasm32-unknown-unknown
cargo xtask test
to run tests.CHANGELOG.md
entry.