-
Notifications
You must be signed in to change notification settings - Fork 269
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
Only return pre-instance functions when instance is NULL #692
Only return pre-instance functions when instance is NULL #692
Conversation
CI Vulkan-Loader build queued with queue ID 39320. |
CI Vulkan-Loader build # 680 running. |
CI Vulkan-Loader build # 680 failed. |
f85a8ca
to
c69b4a4
Compare
CI Vulkan-Loader build queued with queue ID 39329. |
CI Vulkan-Loader build # 681 running. |
CI Vulkan-Loader build # 681 failed. |
CI Vulkan-Loader build queued with queue ID 1126. |
CI Vulkan-Loader build # 758 running. |
CI Vulkan-Loader build # 758 failed. |
vkGetInstanceProcAddr should only return the pre-instance, aka global functions, when the VkInstance handle is NULL. This brings the desktop vulkan loader into alignment on the spec and with the android loader. KhronosGroup/Vulkan-Docs#1605 The repo previously did follow this rule, but was changes in the 1.0.42 header version timeframe for unknown reasons.
c69b4a4
to
3da3bb4
Compare
CI Vulkan-Loader build queued with queue ID 1613. |
CI Vulkan-Loader build # 762 running. |
CI Vulkan-Loader build # 762 passed. |
vkGetInstanceProcAddr
should only return the pre-instance, aka globalfunctions, when the VkInstance handle is NULL. This brings the desktop
vulkan loader into alignment on the spec and with the android loader. See:
KhronosGroup/Vulkan-Docs#1605
This repo previously did follow the rule, but was changed in the
1.0.42 header version timeframe for unknown reasons.
Fixes #692