diff --git a/src/video_core/renderer_vulkan/renderer_vulkan.cpp b/src/video_core/renderer_vulkan/renderer_vulkan.cpp index d25ac17d4..b56c87b71 100644 --- a/src/video_core/renderer_vulkan/renderer_vulkan.cpp +++ b/src/video_core/renderer_vulkan/renderer_vulkan.cpp @@ -183,11 +183,14 @@ static std::array MakeOrthographicMatrix(float width, float height } RendererVulkan::RendererVulkan(Frontend::EmuWindow& window) - : RendererBase{window}, instance{window, Settings::values.physical_device}, - scheduler{instance, *this}, renderpass_cache{instance, scheduler}, - runtime{instance, scheduler, renderpass_cache}, swapchain{instance, renderpass_cache}, - vertex_buffer{ - instance, scheduler, VERTEX_BUFFER_SIZE, vk::BufferUsageFlagBits::eVertexBuffer, {}} { + : RendererBase{window}, instance{window, Settings::values.physical_device}, scheduler{instance, + *this}, + renderpass_cache{instance, scheduler}, runtime{instance, scheduler, renderpass_cache}, + swapchain{instance, renderpass_cache}, vertex_buffer{instance, + scheduler, + VERTEX_BUFFER_SIZE, + vk::BufferUsageFlagBits::eVertexBuffer, + {}} { auto& telemetry_session = Core::System::GetInstance().TelemetrySession(); constexpr auto user_system = Common::Telemetry::FieldType::UserSystem; diff --git a/src/video_core/renderer_vulkan/vk_instance.cpp b/src/video_core/renderer_vulkan/vk_instance.cpp index cffc64787..24db6dbaf 100644 --- a/src/video_core/renderer_vulkan/vk_instance.cpp +++ b/src/video_core/renderer_vulkan/vk_instance.cpp @@ -4,8 +4,8 @@ #include #include "common/assert.h" -#include "core/settings.h" #include "core/frontend/emu_window.h" +#include "core/settings.h" #include "video_core/renderer_vulkan/vk_instance.h" #include "video_core/renderer_vulkan/vk_platform.h" diff --git a/src/video_core/renderer_vulkan/vk_instance.h b/src/video_core/renderer_vulkan/vk_instance.h index 9cf70003f..f01a2ff0a 100644 --- a/src/video_core/renderer_vulkan/vk_instance.h +++ b/src/video_core/renderer_vulkan/vk_instance.h @@ -87,7 +87,6 @@ public: /// Returns true when VK_KHR_timeline_semaphore is supported bool IsTimelineSemaphoreSupported() const { - return false; return timeline_semaphores; } diff --git a/src/video_core/renderer_vulkan/vk_pipeline_cache.cpp b/src/video_core/renderer_vulkan/vk_pipeline_cache.cpp index 68a5a7c97..dcbfda2bb 100644 --- a/src/video_core/renderer_vulkan/vk_pipeline_cache.cpp +++ b/src/video_core/renderer_vulkan/vk_pipeline_cache.cpp @@ -604,8 +604,8 @@ void PipelineCache::BindDescriptorSets() { try { batch = device.allocateDescriptorSets(alloc_info); } catch (vk::OutOfPoolMemoryError& err) { - LOG_CRITICAL(Render_Vulkan, "Run out of pool memory for layout {}: {}", - i, err.what()); + LOG_CRITICAL(Render_Vulkan, "Run out of pool memory for layout {}: {}", i, + err.what()); UNREACHABLE(); } } diff --git a/src/video_core/renderer_vulkan/vk_rasterizer.cpp b/src/video_core/renderer_vulkan/vk_rasterizer.cpp index 14271634b..acf603308 100644 --- a/src/video_core/renderer_vulkan/vk_rasterizer.cpp +++ b/src/video_core/renderer_vulkan/vk_rasterizer.cpp @@ -1589,17 +1589,15 @@ bool RasterizerVulkan::AccelerateDisplay(const GPU::Regs::FramebufferConfig& con vk::Sampler RasterizerVulkan::CreateSampler(const SamplerInfo& info) { const bool use_border_color = instance.IsCustomBorderColorSupported() && - (info.wrap_s == SamplerInfo::TextureConfig::ClampToBorder || - info.wrap_t == SamplerInfo::TextureConfig::ClampToBorder); + (info.wrap_s == SamplerInfo::TextureConfig::ClampToBorder || + info.wrap_t == SamplerInfo::TextureConfig::ClampToBorder); auto properties = instance.GetPhysicalDevice().getProperties(); const auto color = PicaToVK::ColorRGBA8(info.border_color); const vk::SamplerCustomBorderColorCreateInfoEXT border_color_info = { - .customBorderColor = vk::ClearColorValue{ - .float32 = std::array{color[0], color[1], color[2], color[3]} - }, - .format = vk::Format::eUndefined - }; + .customBorderColor = + vk::ClearColorValue{.float32 = std::array{color[0], color[1], color[2], color[3]}}, + .format = vk::Format::eUndefined}; const vk::SamplerCreateInfo sampler_info = { .pNext = use_border_color ? &border_color_info : nullptr, @@ -1615,8 +1613,8 @@ vk::Sampler RasterizerVulkan::CreateSampler(const SamplerInfo& info) { .compareOp = vk::CompareOp::eAlways, .minLod = info.lod_min, .maxLod = info.lod_max, - .borderColor = use_border_color ? vk::BorderColor::eFloatCustomEXT - : vk::BorderColor::eIntOpaqueBlack, + .borderColor = + use_border_color ? vk::BorderColor::eFloatCustomEXT : vk::BorderColor::eIntOpaqueBlack, .unnormalizedCoordinates = false}; vk::Device device = instance.GetDevice(); diff --git a/src/video_core/renderer_vulkan/vk_stream_buffer.cpp b/src/video_core/renderer_vulkan/vk_stream_buffer.cpp index fc5c5e7a3..2b1f7133c 100644 --- a/src/video_core/renderer_vulkan/vk_stream_buffer.cpp +++ b/src/video_core/renderer_vulkan/vk_stream_buffer.cpp @@ -67,9 +67,8 @@ StagingBuffer::~StagingBuffer() { StreamBuffer::StreamBuffer(const Instance& instance, TaskScheduler& scheduler, u32 size, vk::BufferUsageFlagBits usage, std::span view_formats) - : instance{instance}, scheduler{scheduler}, staging{instance, size, - vk::BufferUsageFlagBits::eTransferSrc}, - usage{usage}, total_size{size * SCHEDULER_COMMAND_COUNT} { + : instance{instance}, scheduler{scheduler}, total_size{size * SCHEDULER_COMMAND_COUNT}, + staging{instance, total_size, vk::BufferUsageFlagBits::eTransferSrc}, usage{usage} { const vk::BufferCreateInfo buffer_info = { .size = total_size, .usage = usage | vk::BufferUsageFlagBits::eTransferDst}; diff --git a/src/video_core/renderer_vulkan/vk_stream_buffer.h b/src/video_core/renderer_vulkan/vk_stream_buffer.h index f4038ecb4..5205e4b92 100644 --- a/src/video_core/renderer_vulkan/vk_stream_buffer.h +++ b/src/video_core/renderer_vulkan/vk_stream_buffer.h @@ -75,12 +75,12 @@ private: const Instance& instance; TaskScheduler& scheduler; + u32 total_size = 0; StagingBuffer staging; vk::Buffer buffer{}; VmaAllocation allocation{}; vk::BufferUsageFlagBits usage; - u32 total_size = 0; std::array views{}; std::size_t view_count = 0;