[PR #3484] [MERGED] vk_platform: Remove workaround to use Vulkan loader on macOS. #3506

Closed
opened 2026-02-27 22:03:56 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/shadps4-emu/shadPS4/pull/3484
Author: @squidbus
Created: 8/30/2025
Status: Merged
Merged: 8/30/2025
Merged by: @squidbus

Base: mainHead: remove-vk-workaround


📝 Commits (1)

  • d16cc42 vk_platform: Remove workaround to use Vulkan loader on macOS.

📊 Changes

1 file changed (+2 additions, -12 deletions)

View changed files

📝 src/video_core/renderer_vulkan/vk_platform.cpp (+2 -12)

📄 Description

Previously the loader in Vulkan-Headers would prioritize a loading a bundled libMoltenVK over the Vulkan SDK libvulkan loader in /usr/local/lib, meaning in order to properly use the loader to enable layers we had to work around by checking for and manually supplying the right dylib path.

In newer commits of Vulkan-Headers (which we use already), /usr/local/lib/libvulkan.dylib is now properly prioritized over libMoltenVK.dylib, so we can remove this workaround.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/shadps4-emu/shadPS4/pull/3484 **Author:** [@squidbus](https://github.com/squidbus) **Created:** 8/30/2025 **Status:** ✅ Merged **Merged:** 8/30/2025 **Merged by:** [@squidbus](https://github.com/squidbus) **Base:** `main` ← **Head:** `remove-vk-workaround` --- ### 📝 Commits (1) - [`d16cc42`](https://github.com/shadps4-emu/shadPS4/commit/d16cc4298104718d4d5b609a601e0062d2559df6) vk_platform: Remove workaround to use Vulkan loader on macOS. ### 📊 Changes **1 file changed** (+2 additions, -12 deletions) <details> <summary>View changed files</summary> 📝 `src/video_core/renderer_vulkan/vk_platform.cpp` (+2 -12) </details> ### 📄 Description Previously the loader in `Vulkan-Headers` would prioritize a loading a bundled `libMoltenVK` over the Vulkan SDK `libvulkan` loader in `/usr/local/lib`, meaning in order to properly use the loader to enable layers we had to work around by checking for and manually supplying the right dylib path. In newer commits of `Vulkan-Headers` (which we use already), `/usr/local/lib/libvulkan.dylib` is now properly prioritized over `libMoltenVK.dylib`, so we can remove this workaround. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-27 22:03:56 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/shadPS4#3506
No description provided.