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

Draw indirect count buffer #66

Merged
merged 6 commits into from
Apr 30, 2024

Conversation

vertver
Copy link
Contributor

@vertver vertver commented Apr 29, 2024

No description provided.

@vertver vertver mentioned this pull request Apr 29, 2024
@dzhdanNV dzhdanNV merged commit 8edc018 into NVIDIAGameWorks:main Apr 30, 2024
2 checks passed
@dzhdanNV
Copy link
Collaborator

Thanks! Merged. But actually I'm in a situation, where simpler to accept and review later :)

dzhdanNV added a commit that referenced this pull request Apr 30, 2024
HIGHLIGHTS:
- NRI: fixed compilation issues with Clang
- NRI: exposed "NRICompatibility.hlsli" simplifying cross-API shader compilation
- NRI: introduced "draw count" parameters for "indirect drawing"
- D3D12: added "draw parameters" emulation if requested and not supported (see "PipelineLayoutDesc::enableDrawParametersEmulation")

DETAILS:
- NRI: fixed compilation issues with Clang
- NRI: merged #64
- NRI: merged #66
- NRI: "draw count" functionality merged into "CmdDrawIndirect" and "CmdDrawIndexedIndirect"
- NRI: renamed "enableDrawParametersEmulation" to "enableD3D12DrawParametersEmulation" for readability
- NRI: exposed "DeviceDesc::isDrawIndirectCountSupported"
- NRI: removed (unused?) "DeviceDesc::isRegisterAliasingSupported"
- NRI: "DeviceDesc::isProgrammableSampleLocationsSupported" replaced with "programmableSampleLocationsTier"
- NRI: polished "NRICompatibility.hlsli"
- D3D11: hooked up "indirect draw count" if supported via extensions
- D3D11/D3D12/VK: polished code from PRs
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.

2 participants