diff options
| -rw-r--r-- | src/core/file_sys/patch_manager.cpp | 6 | ||||
| -rw-r--r-- | src/core/hle/service/friend/friend.cpp | 11 | ||||
| -rw-r--r-- | src/video_core/engines/shader_bytecode.h | 6 | ||||
| -rw-r--r-- | src/video_core/renderer_opengl/gl_shader_cache.cpp | 4 | ||||
| -rw-r--r-- | src/video_core/renderer_opengl/gl_shader_decompiler.cpp | 6 | ||||
| -rw-r--r-- | src/video_core/renderer_opengl/gl_texture_cache.cpp | 2 | ||||
| -rw-r--r-- | src/video_core/renderer_vulkan/maxwell_to_vk.cpp | 10 | ||||
| -rw-r--r-- | src/video_core/renderer_vulkan/vk_blit_screen.cpp | 2 | ||||
| -rw-r--r-- | src/video_core/renderer_vulkan/vk_rasterizer.cpp | 9 | ||||
| -rw-r--r-- | src/video_core/shader/decode/arithmetic.cpp | 3 | ||||
| -rw-r--r-- | src/video_core/texture_cache/texture_cache.h | 4 | 
11 files changed, 51 insertions, 12 deletions
| diff --git a/src/core/file_sys/patch_manager.cpp b/src/core/file_sys/patch_manager.cpp index e226e9711..e77e82b8d 100644 --- a/src/core/file_sys/patch_manager.cpp +++ b/src/core/file_sys/patch_manager.cpp @@ -348,6 +348,12 @@ static void ApplyLayeredFS(VirtualFile& romfs, u64 title_id, ContentRecordType t          if (ext_dir != nullptr)              layers_ext.push_back(std::move(ext_dir));      } + +    // When there are no layers to apply, return early as there is no need to rebuild the RomFS +    if (layers.empty() && layers_ext.empty()) { +        return; +    } +      layers.push_back(std::move(extracted));      auto layered = LayeredVfsDirectory::MakeLayeredDirectory(std::move(layers)); diff --git a/src/core/hle/service/friend/friend.cpp b/src/core/hle/service/friend/friend.cpp index 6aadb3ea8..7938b4b80 100644 --- a/src/core/hle/service/friend/friend.cpp +++ b/src/core/hle/service/friend/friend.cpp @@ -27,7 +27,7 @@ public:              {10110, nullptr, "GetFriendProfileImage"},              {10200, nullptr, "SendFriendRequestForApplication"},              {10211, nullptr, "AddFacedFriendRequestForApplication"}, -            {10400, nullptr, "GetBlockedUserListIds"}, +            {10400, &IFriendService::GetBlockedUserListIds, "GetBlockedUserListIds"},              {10500, nullptr, "GetProfileList"},              {10600, nullptr, "DeclareOpenOnlinePlaySession"},              {10601, &IFriendService::DeclareCloseOnlinePlaySession, "DeclareCloseOnlinePlaySession"}, @@ -121,6 +121,15 @@ private:      };      static_assert(sizeof(SizedFriendFilter) == 0x10, "SizedFriendFilter is an invalid size"); +    void GetBlockedUserListIds(Kernel::HLERequestContext& ctx) { +        // This is safe to stub, as there should be no adverse consequences from reporting no +        // blocked users. +        LOG_WARNING(Service_ACC, "(STUBBED) called"); +        IPC::ResponseBuilder rb{ctx, 3}; +        rb.Push(RESULT_SUCCESS); +        rb.Push<u32>(0); // Indicates there are no blocked users +    } +      void DeclareCloseOnlinePlaySession(Kernel::HLERequestContext& ctx) {          // Stub used by Splatoon 2          LOG_WARNING(Service_ACC, "(STUBBED) called"); diff --git a/src/video_core/engines/shader_bytecode.h b/src/video_core/engines/shader_bytecode.h index c66c66f6c..430ee86ba 100644 --- a/src/video_core/engines/shader_bytecode.h +++ b/src/video_core/engines/shader_bytecode.h @@ -1871,7 +1871,8 @@ public:          ICMP_R,          ICMP_CR,          ICMP_IMM, -        FCMP_R, +        FCMP_RR, +        FCMP_RC,          MUFU,  // Multi-Function Operator          RRO_C, // Range Reduction Operator          RRO_R, @@ -2179,7 +2180,8 @@ private:              INST("0101110100100---", Id::HSETP2_R, Type::HalfSetPredicate, "HSETP2_R"),              INST("0111111-0-------", Id::HSETP2_IMM, Type::HalfSetPredicate, "HSETP2_IMM"),              INST("0101110100011---", Id::HSET2_R, Type::HalfSet, "HSET2_R"), -            INST("010110111010----", Id::FCMP_R, Type::Arithmetic, "FCMP_R"), +            INST("010110111010----", Id::FCMP_RR, Type::Arithmetic, "FCMP_RR"), +            INST("010010111010----", Id::FCMP_RC, Type::Arithmetic, "FCMP_RC"),              INST("0101000010000---", Id::MUFU, Type::Arithmetic, "MUFU"),              INST("0100110010010---", Id::RRO_C, Type::Arithmetic, "RRO_C"),              INST("0101110010010---", Id::RRO_R, Type::Arithmetic, "RRO_R"), diff --git a/src/video_core/renderer_opengl/gl_shader_cache.cpp b/src/video_core/renderer_opengl/gl_shader_cache.cpp index 6d2ff20f9..12c6dcfde 100644 --- a/src/video_core/renderer_opengl/gl_shader_cache.cpp +++ b/src/video_core/renderer_opengl/gl_shader_cache.cpp @@ -34,6 +34,8 @@  namespace OpenGL {  using Tegra::Engines::ShaderType; +using VideoCommon::Shader::CompileDepth; +using VideoCommon::Shader::CompilerSettings;  using VideoCommon::Shader::ProgramCode;  using VideoCommon::Shader::Registry;  using VideoCommon::Shader::ShaderIR; @@ -43,7 +45,7 @@ namespace {  constexpr u32 STAGE_MAIN_OFFSET = 10;  constexpr u32 KERNEL_MAIN_OFFSET = 0; -constexpr VideoCommon::Shader::CompilerSettings COMPILER_SETTINGS{}; +constexpr CompilerSettings COMPILER_SETTINGS{CompileDepth::FullDecompile};  /// Gets the address for the specified shader stage program  GPUVAddr GetShaderAddress(Core::System& system, Maxwell::ShaderProgram program) { diff --git a/src/video_core/renderer_opengl/gl_shader_decompiler.cpp b/src/video_core/renderer_opengl/gl_shader_decompiler.cpp index 1f1f01313..59bbd1211 100644 --- a/src/video_core/renderer_opengl/gl_shader_decompiler.cpp +++ b/src/video_core/renderer_opengl/gl_shader_decompiler.cpp @@ -1821,13 +1821,15 @@ private:      Expression HMergeH0(Operation operation) {          const std::string dest = VisitOperand(operation, 0).AsUint();          const std::string src = VisitOperand(operation, 1).AsUint(); -        return {fmt::format("bitfieldInsert({}, {}, 0, 16)", dest, src), Type::Uint}; +        return {fmt::format("vec2(unpackHalf2x16({}).x, unpackHalf2x16({}).y)", src, dest), +                Type::HalfFloat};      }      Expression HMergeH1(Operation operation) {          const std::string dest = VisitOperand(operation, 0).AsUint();          const std::string src = VisitOperand(operation, 1).AsUint(); -        return {fmt::format("bitfieldInsert({}, {}, 16, 16)", dest, src), Type::Uint}; +        return {fmt::format("vec2(unpackHalf2x16({}).x, unpackHalf2x16({}).y)", dest, src), +                Type::HalfFloat};      }      Expression HPack2(Operation operation) { diff --git a/src/video_core/renderer_opengl/gl_texture_cache.cpp b/src/video_core/renderer_opengl/gl_texture_cache.cpp index 0b4d999d7..2729d1265 100644 --- a/src/video_core/renderer_opengl/gl_texture_cache.cpp +++ b/src/video_core/renderer_opengl/gl_texture_cache.cpp @@ -417,7 +417,7 @@ void CachedSurfaceView::Attach(GLenum attachment, GLenum target) const {          switch (params.target) {          case SurfaceTarget::Texture2DArray: -            glFramebufferTexture(target, attachment, GetTexture(), params.base_level); +            glFramebufferTexture(target, attachment, GetTexture(), 0);              break;          default:              UNIMPLEMENTED(); diff --git a/src/video_core/renderer_vulkan/maxwell_to_vk.cpp b/src/video_core/renderer_vulkan/maxwell_to_vk.cpp index 143478863..8681b821f 100644 --- a/src/video_core/renderer_vulkan/maxwell_to_vk.cpp +++ b/src/video_core/renderer_vulkan/maxwell_to_vk.cpp @@ -360,6 +360,7 @@ VkFormat VertexFormat(Maxwell::VertexAttribute::Type type, Maxwell::VertexAttrib          default:              break;          } +        break;      case Maxwell::VertexAttribute::Type::UnsignedInt:          switch (size) {          case Maxwell::VertexAttribute::Size::Size_8: @@ -370,6 +371,14 @@ VkFormat VertexFormat(Maxwell::VertexAttribute::Type type, Maxwell::VertexAttrib              return VK_FORMAT_R8G8B8_UINT;          case Maxwell::VertexAttribute::Size::Size_8_8_8_8:              return VK_FORMAT_R8G8B8A8_UINT; +        case Maxwell::VertexAttribute::Size::Size_16: +            return VK_FORMAT_R16_UINT; +        case Maxwell::VertexAttribute::Size::Size_16_16: +            return VK_FORMAT_R16G16_UINT; +        case Maxwell::VertexAttribute::Size::Size_16_16_16: +            return VK_FORMAT_R16G16B16_UINT; +        case Maxwell::VertexAttribute::Size::Size_16_16_16_16: +            return VK_FORMAT_R16G16B16A16_UINT;          case Maxwell::VertexAttribute::Size::Size_32:              return VK_FORMAT_R32_UINT;          case Maxwell::VertexAttribute::Size::Size_32_32: @@ -381,6 +390,7 @@ VkFormat VertexFormat(Maxwell::VertexAttribute::Type type, Maxwell::VertexAttrib          default:              break;          } +        break;      case Maxwell::VertexAttribute::Type::UnsignedScaled:          switch (size) {          case Maxwell::VertexAttribute::Size::Size_8: diff --git a/src/video_core/renderer_vulkan/vk_blit_screen.cpp b/src/video_core/renderer_vulkan/vk_blit_screen.cpp index 21644a7e7..fbd406f2b 100644 --- a/src/video_core/renderer_vulkan/vk_blit_screen.cpp +++ b/src/video_core/renderer_vulkan/vk_blit_screen.cpp @@ -535,7 +535,9 @@ void VKBlitScreen::CreateGraphicsPipeline() {      viewport_state_ci.pNext = nullptr;      viewport_state_ci.flags = 0;      viewport_state_ci.viewportCount = 1; +    viewport_state_ci.pViewports = nullptr;      viewport_state_ci.scissorCount = 1; +    viewport_state_ci.pScissors = nullptr;      VkPipelineRasterizationStateCreateInfo rasterization_ci;      rasterization_ci.sType = VK_STRUCTURE_TYPE_PIPELINE_RASTERIZATION_STATE_CREATE_INFO; diff --git a/src/video_core/renderer_vulkan/vk_rasterizer.cpp b/src/video_core/renderer_vulkan/vk_rasterizer.cpp index 33cbc0bb6..774ba1f26 100644 --- a/src/video_core/renderer_vulkan/vk_rasterizer.cpp +++ b/src/video_core/renderer_vulkan/vk_rasterizer.cpp @@ -62,13 +62,16 @@ constexpr auto ComputeShaderIndex = static_cast<std::size_t>(Tegra::Engines::Sha  VkViewport GetViewportState(const VKDevice& device, const Maxwell& regs, std::size_t index) {      const auto& src = regs.viewport_transform[index]; +    const float width = src.scale_x * 2.0f; +    const float height = src.scale_y * 2.0f; +      VkViewport viewport;      viewport.x = src.translate_x - src.scale_x;      viewport.y = src.translate_y - src.scale_y; -    viewport.width = src.scale_x * 2.0f; -    viewport.height = src.scale_y * 2.0f; +    viewport.width = width != 0.0f ? width : 1.0f; +    viewport.height = height != 0.0f ? height : 1.0f; -    const float reduce_z = regs.depth_mode == Maxwell::DepthMode::MinusOneToOne; +    const float reduce_z = regs.depth_mode == Maxwell::DepthMode::MinusOneToOne ? 1.0f : 0.0f;      viewport.minDepth = src.translate_z - src.scale_z * reduce_z;      viewport.maxDepth = src.translate_z + src.scale_z;      if (!device.IsExtDepthRangeUnrestrictedSupported()) { diff --git a/src/video_core/shader/decode/arithmetic.cpp b/src/video_core/shader/decode/arithmetic.cpp index 478394682..4db329fa5 100644 --- a/src/video_core/shader/decode/arithmetic.cpp +++ b/src/video_core/shader/decode/arithmetic.cpp @@ -136,7 +136,8 @@ u32 ShaderIR::DecodeArithmetic(NodeBlock& bb, u32 pc) {          SetRegister(bb, instr.gpr0, value);          break;      } -    case OpCode::Id::FCMP_R: { +    case OpCode::Id::FCMP_RR: +    case OpCode::Id::FCMP_RC: {          UNIMPLEMENTED_IF(instr.fcmp.ftz == 0);          Node op_c = GetRegister(instr.gpr39);          Node comp = GetPredicateComparisonFloat(instr.fcmp.cond, std::move(op_c), Immediate(0.0f)); diff --git a/src/video_core/texture_cache/texture_cache.h b/src/video_core/texture_cache/texture_cache.h index cfc7fe6e9..4edd4313b 100644 --- a/src/video_core/texture_cache/texture_cache.h +++ b/src/video_core/texture_cache/texture_cache.h @@ -509,7 +509,9 @@ private:          }          const auto& final_params = new_surface->GetSurfaceParams();          if (cr_params.type != final_params.type) { -            BufferCopy(current_surface, new_surface); +            if (Settings::values.use_accurate_gpu_emulation) { +                BufferCopy(current_surface, new_surface); +            }          } else {              std::vector<CopyParams> bricks = current_surface->BreakDown(final_params);              for (auto& brick : bricks) { | 
