diff --git a/src/common/serdes.h b/src/common/serdes.h index f91a0ace8..a8152e95a 100644 --- a/src/common/serdes.h +++ b/src/common/serdes.h @@ -42,7 +42,8 @@ struct Archive { } void Advance(size_t size) { - ASSERT(offset + size <= container.size()); + ASSERT_MSG(offset + size <= container.size(), + "Invalid or corrupted deserialization container/shader cache"); offset += size; } @@ -104,7 +105,8 @@ struct Writer { struct Reader { template void Read(T* ptr, size_t size) { - ASSERT(ar.offset + size <= ar.container.size()); + ASSERT_MSG(ar.offset + size <= ar.container.size(), + "Invalid or corrupted deserialization container/shader cache"); std::memcpy(reinterpret_cast(ptr), ar.CurrPtr(), size); ar.Advance(size); } diff --git a/src/imgui/renderer/imgui_impl_vulkan.cpp b/src/imgui/renderer/imgui_impl_vulkan.cpp index 97f44c318..0ef1d263d 100644 --- a/src/imgui/renderer/imgui_impl_vulkan.cpp +++ b/src/imgui/renderer/imgui_impl_vulkan.cpp @@ -1219,6 +1219,10 @@ void ImGuiImplVulkanDestroyDeviceObjects() { v.device.destroyDescriptorSetLayout(bd->descriptor_set_layout, v.allocator); bd->descriptor_set_layout = VK_NULL_HANDLE; } + if (bd->descriptor_pool) { + v.device.destroyDescriptorPool(bd->descriptor_pool, v.allocator); + bd->descriptor_pool = VK_NULL_HANDLE; + } if (bd->pipeline_layout) { v.device.destroyPipelineLayout(bd->pipeline_layout, v.allocator); bd->pipeline_layout = VK_NULL_HANDLE; diff --git a/src/shader_recompiler/frontend/copy_shader.cpp b/src/shader_recompiler/frontend/copy_shader.cpp index 795003e43..ff3045b86 100644 --- a/src/shader_recompiler/frontend/copy_shader.cpp +++ b/src/shader_recompiler/frontend/copy_shader.cpp @@ -45,6 +45,14 @@ CopyShaderData ParseCopyShader(std::span code) { sources[inst.dst[0].code] += inst.control.sopk.simm; break; } + case Gcn::Opcode::S_BFM_B32: { + ASSERT(inst.src[0].field == Gcn::OperandField::SignedConstIntPos && + inst.src[1].field == Gcn::OperandField::SignedConstIntPos); + const auto src0 = inst.src[0].code - Gcn::OperandFieldRange::SignedConstIntPosMin + 1; + const auto src1 = inst.src[1].code - Gcn::OperandFieldRange::SignedConstIntPosMin + 1; + sources[inst.dst[0].code] = ((1 << src0) - 1) << src1; + break; + } case Gcn::Opcode::EXP: { const auto& exp = inst.control.exp; const IR::Attribute semantic = static_cast(exp.target); @@ -71,6 +79,7 @@ CopyShaderData ParseCopyShader(std::span code) { ASSERT(sources[index] != -1); offsets[inst.src[1].code] += sources[index]; } + data.num_comps++; break; } default: diff --git a/src/shader_recompiler/frontend/copy_shader.h b/src/shader_recompiler/frontend/copy_shader.h index 24c7060ed..c416bde99 100644 --- a/src/shader_recompiler/frontend/copy_shader.h +++ b/src/shader_recompiler/frontend/copy_shader.h @@ -15,6 +15,7 @@ struct CopyShaderData { std::map> attr_map; u32 num_attrs{0}; u32 output_vertices{0}; + u32 num_comps{0}; }; CopyShaderData ParseCopyShader(std::span code); diff --git a/src/shader_recompiler/ir/passes/ring_access_elimination.cpp b/src/shader_recompiler/ir/passes/ring_access_elimination.cpp index f8818b622..a7a23a726 100644 --- a/src/shader_recompiler/ir/passes/ring_access_elimination.cpp +++ b/src/shader_recompiler/ir/passes/ring_access_elimination.cpp @@ -104,6 +104,13 @@ void RingAccessElimination(const IR::Program& program, const RuntimeInfo& runtim output_vertices, info.gs_copy_data.output_vertices); output_vertices = info.gs_copy_data.output_vertices; } + u32 dwords_per_vertex = gs_info.out_vertex_data_size; + if (info.gs_copy_data.num_comps && info.gs_copy_data.num_comps != dwords_per_vertex) { + LOG_WARNING(Render_Vulkan, + "VERT_ITEMSIZE {} is different than actual number of dwords per vertex {}", + dwords_per_vertex, info.gs_copy_data.num_comps); + dwords_per_vertex = info.gs_copy_data.num_comps; + } ForEachInstruction([&](IR::IREmitter& ir, IR::Inst& inst) { const auto opcode = inst.GetOpcode(); @@ -139,7 +146,7 @@ void RingAccessElimination(const IR::Program& program, const RuntimeInfo& runtim const auto offset = inst.Flags().inst_offset.Value(); const auto data = ir.BitCast(IR::U32{inst.Arg(2)}); const auto comp_ofs = output_vertices * 4u; - const auto output_size = comp_ofs * gs_info.out_vertex_data_size; + const auto output_size = comp_ofs * dwords_per_vertex; const auto vc_read_ofs = (((offset / comp_ofs) * comp_ofs) % output_size) * 16u; const auto& it = info.gs_copy_data.attr_map.find(vc_read_ofs); diff --git a/src/video_core/renderer_vulkan/vk_swapchain.cpp b/src/video_core/renderer_vulkan/vk_swapchain.cpp index 04f9d8504..4904319bd 100644 --- a/src/video_core/renderer_vulkan/vk_swapchain.cpp +++ b/src/video_core/renderer_vulkan/vk_swapchain.cpp @@ -261,6 +261,12 @@ void Swapchain::Destroy() { LOG_WARNING(Render_Vulkan, "Failed to wait for device to become idle: {}", vk::to_string(wait_result)); } + + for (auto& image_view : images_view) { + device.destroyImageView(image_view); + } + images_view.clear(); + if (swapchain) { device.destroySwapchainKHR(swapchain); }