Skip to content

Conversation

@teoxoy
Copy link
Member

@teoxoy teoxoy commented Apr 14, 2025

Connections
Closes #2471

Description
Gets the vertex_index & instance_index builtins working for indirect draws by updating the existing root constants (using the validation shader; same strategy as in #5730).

Testing
Existing wgpu_gpu::vertex_indices test.

Squash or Rebase?
Rebase

@teoxoy teoxoy requested a review from a team as a code owner April 14, 2025 14:43
@ErichDonGubler ErichDonGubler self-assigned this Apr 14, 2025
@teoxoy teoxoy force-pushed the indirect-draw-builtins branch from 77fa219 to d1d3df7 Compare April 14, 2025 15:28
Copy link
Member

@ErichDonGubler ErichDonGubler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving conditional on resolution of questions.

@teoxoy teoxoy force-pushed the indirect-draw-builtins branch from d1d3df7 to ac04c6e Compare April 15, 2025 12:17
@teoxoy teoxoy force-pushed the indirect-draw-builtins branch from ac04c6e to ac1a413 Compare April 15, 2025 12:21
@teoxoy teoxoy merged commit 62d4e82 into gfx-rs:trunk Apr 15, 2025
37 checks passed
@teoxoy teoxoy deleted the indirect-draw-builtins branch April 15, 2025 15:57
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

Successfully merging this pull request may close these issues.

[D3D12] Fix vertex_index and instance_index built-in inputs for indirect draw calls

3 participants