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

Fixed VkResult Error on macOS VulkanSDK 1.3.216 #6172

Closed
wants to merge 1 commit into from

Conversation

NewSkyLine-dev
Copy link

@NewSkyLine-dev NewSkyLine-dev commented Feb 16, 2023

@omar, @zao, @wolfpld I accidentally closed last PR. Here is the update version of #6109

Added function is_instance_extension_enabled() to check if instance functions are enabled. Helped by adding the enumeration extension
ocornut added a commit that referenced this pull request Apr 13, 2023
…dation layer errors. (#6109, #6172, #6101)

Enable VK_KHR_GET_PHYSICAL_DEVICE_PROPERTIES_2_EXTENSION_NAME, VK_KHR_PORTABILITY_ENUMERATION_EXTENSION_NAME, VK_INSTANCE_CREATE_ENUMERATE_PORTABILITY_BIT_KHR
@ocornut
Copy link
Owner

ocornut commented Apr 13, 2023

It turns out that your PR still breaks debug build (with IMGUI_VULKAN_DEBUG_REPORT)
I reworked this based on this and Bartosz's feedback on email with this commit: 6324280
Fixing validation errors for me that I got with latest SDK.
Thanks!

@ocornut ocornut closed this Apr 13, 2023
ocornut added a commit that referenced this pull request Apr 13, 2023
@NewSkyLine-dev
Copy link
Author

Happy that I could help, even if it wasn't much ☺️

kjblanchard pushed a commit to kjblanchard/imgui that referenced this pull request May 5, 2023
…dation layer errors. (ocornut#6109, ocornut#6172, ocornut#6101)

Enable VK_KHR_GET_PHYSICAL_DEVICE_PROPERTIES_2_EXTENSION_NAME, VK_KHR_PORTABILITY_ENUMERATION_EXTENSION_NAME, VK_INSTANCE_CREATE_ENUMERATE_PORTABILITY_BIT_KHR
kjblanchard pushed a commit to kjblanchard/imgui that referenced this pull request May 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants