Skip to content

Conversation

jonjondev
Copy link
Member

Description

This PR upgrades Vulkan across macOS, Windows, and Linux to v1.4.321.0 alongide the MoltenVK upgrade to v1.4.0. Some scripts have been updated to use the full version tag, and the Volk step in the setup scripts have been pinned to the correct Vulkan version.

The PR has been...

  • provided a reasonable name that is not just the branch name (e.g "Added Vulkan render delegate")
  • linked to its related issue
  • assigned a reviewer from the team
  • labelled appropriately

The code has been...

  • made mergable and free of conflicts in relation to master (according to GitHub)
  • tested in a packaged state using the package targets
  • pulled to the reviewer's machine and reasonably tested

@jonjondev jonjondev requested a review from Raelr August 21, 2025 21:57
@jonjondev jonjondev added the build Relating to the engine's build system label Aug 21, 2025
@@ -281,7 +282,7 @@ setup_moltenVk() {
checkout_tags "${VULKAN_VENDOR_DIR}"/MoltenVK ${MOLTENVK_VERSION}

echo "Building MoltenVk..."
(cd "${VULKAN_VENDOR_DIR}"/MoltenVK ; ./fetchDependencies --macos --v-headers-root "${VULKAN_VENDOR_DIR}"/Vulkan-Headers --glslang-root "${VENDOR_DIR}"/glslang)
(cd "${VULKAN_VENDOR_DIR}"/MoltenVK ; ./fetchDependencies --macos)
Copy link
Member Author

Choose a reason for hiding this comment

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

@Raelr it seems the fetchDependencies script has been changed and the --glslang-root argument has been removed. The only arguments it now supports is --spirv-tools-root <path>, --build-spirv-tools, --spirv-cross-root <path>, and --v-headers-root <path>. Providing any combination of those arguments results in compilation errors. This should probably be looked into before we proceed.

Otherwise everything seems to build and compile with with and without validation layers enabled.

@jonjondev jonjondev force-pushed the feature/vulkan-upgrade branch from b1204c1 to 860ff24 Compare August 21, 2025 23:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Relating to the engine's build system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant