radv: Remove redundant radv_QueuePresentKHR.

Does the same thing as the wsi function.

Reviewed-by: Samuel Pitoiset <samuel.pitoiset@gmail.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/18293>
This commit is contained in:
Bas Nieuwenhuizen 2022-08-29 13:21:59 +02:00 committed by Marge Bot
parent 0e0e5d1f20
commit 40ad12126f
4 changed files with 7 additions and 11 deletions

View file

@ -22,6 +22,7 @@
*/
#include "vk_common_entrypoints.h"
#include "wsi_common_entrypoints.h"
#include "radv_private.h"
#include "radv_shader.h"
@ -404,7 +405,7 @@ sqtt_QueuePresentKHR(VkQueue _queue, const VkPresentInfoKHR *pPresentInfo)
{
VkResult result;
result = radv_QueuePresentKHR(_queue, pPresentInfo);
result = wsi_QueuePresentKHR(_queue, pPresentInfo);
if (result != VK_SUCCESS)
return result;

View file

@ -177,7 +177,7 @@ libvulkan_radeon = shared_library(
dep_valgrind, radv_deps, idep_aco,
idep_mesautil, idep_nir, idep_vulkan_util, idep_vulkan_wsi,
idep_vulkan_runtime, idep_amdgfxregs_h, idep_xmlconfig,
idep_vulkan_common_entrypoints_h
idep_vulkan_common_entrypoints_h, idep_vulkan_wsi_entrypoints_h
],
c_args : [no_override_init_args, radv_flags, c_msvc_compat_args],
cpp_args : [radv_flags, cpp_msvc_compat_args],

View file

@ -113,12 +113,3 @@ radv_finish_wsi(struct radv_physical_device *physical_device)
physical_device->vk.wsi_device = NULL;
wsi_device_finish(&physical_device->wsi_device, &physical_device->instance->vk.alloc);
}
VKAPI_ATTR VkResult VKAPI_CALL
radv_QueuePresentKHR(VkQueue _queue, const VkPresentInfoKHR *pPresentInfo)
{
RADV_FROM_HANDLE(radv_queue, queue, _queue);
return wsi_common_queue_present(&queue->device->physical_device->wsi_device,
radv_device_to_handle(queue->device), _queue,
queue->vk.queue_family_index, pPresentInfo);
}

View file

@ -71,6 +71,10 @@ idep_vulkan_wsi_headers = declare_dependency(
include_directories : include_directories('.')
)
idep_vulkan_wsi_entrypoints_h = declare_dependency(
sources : [wsi_entrypoints[0]]
)
# This is likely a bug in the Meson VS backend, as MSVC with ninja works fine.
# See this discussion here:
# https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/10506