Vulkan: Update all components to Vulkan SDK 1.4.313.0

[akien-mga]

Needs to be tested on all platforms where we support Vulkan, but also D3D12 and Metal for SPIR-V.
Our last sync was a year ago so this is long overdue, and thus would be good to include in 4.5.

Updated:

• vulkan headers
• volk
• glslang
• spirv-reflect
• spirv-cross
• vk_mem_alloc 3.3.0

[akien-mga]

I disabled this to solve these build errors:

 Error: D:\a\godot\godot\thirdparty\vulkan\vk_mem_alloc.h(6217): error C2039: 'CloseHandle': is not a member of '`global namespace''
Error: D:\a\godot\godot\thirdparty\vulkan\vk_mem_alloc.h(6217): error C3861: 'CloseHandle': identifier not found
Error: D:\a\godot\godot\thirdparty\vulkan\vk_mem_alloc.h(6267): error C2039: 'GetCurrentProcess': is not a member of '`global namespace''
Error: D:\a\godot\godot\thirdparty\vulkan\vk_mem_alloc.h(6267): error C3861: 'GetCurrentProcess': identifier not found
Error: D:\a\godot\godot\thirdparty\vulkan\vk_mem_alloc.h(6269): error C2039: 'DuplicateHandle': is not a member of '`global namespace''
Error: D:\a\godot\godot\thirdparty\vulkan\vk_mem_alloc.h(6269): error C2065: 'FALSE': undeclared identifier
Error: D:\a\godot\godot\thirdparty\vulkan\vk_mem_alloc.h(6269): error C2065: 'DUPLICATE_SAME_ACCESS': undeclared identifier
Error: D:\a\godot\godot\thirdparty\vulkan\vk_mem_alloc.h(6269): error C3861: 'DuplicateHandle': identifier not found

AFAIK we don't use this extension, it's just automatically picked up as present in the Vulkan headers. It was added in VMA 3.2.0: https://github.com/GPUOpen-LibrariesAndSDKs/VulkanMemoryAllocator/blob/master/CHANGELOG.md#320-2024-12-30

If we want to use it, we can likely solve the issues by including windows.h ( 😿 ) or a narrower Win32 header.

[stuartcarnie]

Metal fix looks good to me, as noted, @akien-mga added an additional case to handle a new enum variant. We can ignore the variant, as it is specifically for this extension, which Godot doesn't use. If that were to change, we'd need to update this code

[akien-mga]

BTW, I noticed this in drivers/vulkan/SCsub:

godot/drivers/vulkan/SCsub

Lines 43 to 49 in 48f361a

	elif env["platform"] == "android":
	# Our current NDK version only provides old Vulkan headers,
	# so we have to limit VMA.
	env_thirdparty_vma.AppendUnique(CPPDEFINES=["VMA_VULKAN_VERSION=1000000"])
	elif env["platform"] == "macos" or env["platform"] == "ios":
	# MoltenVK supports only Vulkan 1.1 API, limit VMA to the same version.
	env_thirdparty_vma.AppendUnique(CPPDEFINES=["VMA_VULKAN_VERSION=1001000"])

The Android note is likely obsolete now that we're using NDK r28, we might want to reconsider.
Likewise the note for macOS and iOS could be checked against the latest MoltenVK to see if it's still relevant.
I don't know what's the actual impact of "limiting VMA" to specific Vulkan versions though.

[bruvzg]

I'm getting a lot of pipeline errors with this PR when running tps-demo using Vulkan on macOS (but the demo seems to be running fine).

ERROR: The shader in this pipeline requires a push constant to be set before drawing, but it's not present.
   at: compute_list_dispatch (servers/rendering/rendering_device.cpp:5275)
ERROR: This compute pipeline requires (20) bytes of push constant data, supplied: (32)
   at: compute_list_set_push_constant (servers/rendering/rendering_device.cpp:5236)

ERROR: The shader in this pipeline requires a push constant to be set before drawing, but it's not present.
   at: compute_list_dispatch (servers/rendering/rendering_device.cpp:5275)
ERROR: This compute pipeline requires (8) bytes of push constant data, supplied: (16)
   at: compute_list_set_push_constant (servers/rendering/rendering_device.cpp:5236)

[dsnopek]

I'm getting the same errors as @bruvzg spammed super fast, to the point that I can't seem to operate the editor built from this PR. At least I think it's due to the error spam and not some lower-level issue, but I'm not sure

But if I use an editor compiled from master, and an Android template built from this PR deployed to a Meta Quest 3, the app is usable (still with tons of error spam, but it works), however, the colors look darker than they should be

[akien-mga]

I won't have time to debug the pipeline errors as I'm going on leave :D

If we'd still like to do this update for 4.5 (or whichever SDK comes next, they've been tagging versions weekly for a month now, I expect one will soon be promoted to SDK), someone else will have to take over.