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

Fix sync2 enumerate device extension #304

Merged
merged 2 commits into from
Aug 24, 2023
Merged

Conversation

ziga-lunarg
Copy link
Collaborator

@ziga-lunarg ziga-lunarg commented Aug 24, 2023

Closes #303

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ExtensionLayer build queued with queue ID 29544.

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ExtensionLayer build # 683 running.

Copy link
Contributor

@christophe-lunarg christophe-lunarg left a comment

Choose a reason for hiding this comment

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

This looks great to me!

Copy link
Collaborator

@jeremyg-lunarg jeremyg-lunarg left a comment

Choose a reason for hiding this comment

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

LGTM. Nice test case

@ci-tester-lunarg
Copy link
Collaborator

CI Vulkan-ExtensionLayer build # 683 passed.

@ziga-lunarg ziga-lunarg merged commit da22040 into main Aug 24, 2023
21 checks passed
@ziga-lunarg ziga-lunarg deleted the ziga-fix-sync2-enumerate branch August 24, 2023 17:20
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.

When using sync2 layer, VK_KHR_synchronization2 extension is reported twice
4 participants