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 VK_KHR_draw_indirect_count extensions. #2661

Merged
merged 1 commit into from
Mar 8, 2019

Conversation

AWoloszyn
Copy link
Contributor

This also adds VK_AMD_draw_indirect_count which is the same.

@AWoloszyn AWoloszyn requested review from Qining and pmuetschard March 7, 2019 19:00
@AWoloszyn AWoloszyn force-pushed the VK_KHR_draw_indirect_count branch 2 times, most recently from a0c1d60 to 958f567 Compare March 7, 2019 19:17
return nil, nil, fmt.Errorf("Cannot find Buffer %v", d.Buffer())
}
if !GetState(s).Buffers().Contains(d.CountBuffer()) {
return nil, nil, fmt.Errorf("Cannot find Buffer %v", d.Buffer())
Copy link
Member

Choose a reason for hiding this comment

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

It might be helpful to say "Count Buffer" here and in each of the other functions, so that it's more clear which argument the error is for.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@@ -227,6 +229,9 @@ sub ref!ExtensionSet supportedDeviceExtensions() {
supported.ExtensionNames["VK_KHR_maintenance1"] = true
supported.ExtensionNames["VK_KHR_maintenance2"] = true
supported.ExtensionNames["VK_KHR_maintenance3"] = true
supported.ExtensionNames["VK_KHR_maintenance3"] = true
Copy link
Member

Choose a reason for hiding this comment

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

This one is extra and duplicates the previous line.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done


sub void dovkCmdDrawIndirectCountCOMMON!T(T draw) {
useRenderPass()
if draw.MaxDrawCount > 0 {
Copy link
Member

Choose a reason for hiding this comment

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

nit: Everywhere this sub is called, it's already guarded with this check. Could make it a precondition of this sub - I think a comment would do.

Same for dovkCmdDrawIndexedIndirectCountCOMMON

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

This also adds VK_AMD_draw_indirect_count which is the same.
@AWoloszyn AWoloszyn force-pushed the VK_KHR_draw_indirect_count branch from 958f567 to 94f99e0 Compare March 8, 2019 15:31
@AWoloszyn AWoloszyn merged commit d4bf475 into google:master Mar 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants