diff options
| -rw-r--r-- | src/audio_core/algorithm/filter.cpp | 2 | ||||
| -rw-r--r-- | src/core/file_sys/vfs.cpp | 1 | ||||
| -rw-r--r-- | src/core/file_sys/vfs.h | 5 | ||||
| -rw-r--r-- | src/core/hle/service/am/am.cpp | 7 | ||||
| -rw-r--r-- | src/core/hle/service/filesystem/filesystem.cpp | 2 | ||||
| -rw-r--r-- | src/core/hle/service/filesystem/filesystem.h | 2 | ||||
| -rw-r--r-- | src/core/hle/service/filesystem/fsp_srv.cpp | 1 | ||||
| -rw-r--r-- | src/core/hle/service/ns/pl_u.cpp | 166 | ||||
| -rw-r--r-- | src/video_core/engines/shader_bytecode.h | 2 | ||||
| -rw-r--r-- | src/video_core/renderer_opengl/gl_shader_decompiler.cpp | 134 | ||||
| -rw-r--r-- | src/video_core/renderer_opengl/gl_shader_gen.h | 50 | 
11 files changed, 185 insertions, 187 deletions
| diff --git a/src/audio_core/algorithm/filter.cpp b/src/audio_core/algorithm/filter.cpp index 403b8503f..9fcd0614d 100644 --- a/src/audio_core/algorithm/filter.cpp +++ b/src/audio_core/algorithm/filter.cpp @@ -46,7 +46,7 @@ void Filter::Process(std::vector<s16>& signal) {              out[0][ch] = b0 * in[0][ch] + b1 * in[1][ch] + b2 * in[2][ch] - a1 * out[1][ch] -                           a2 * out[2][ch]; -            signal[i * 2 + ch] = std::clamp(out[0][ch], -32768.0, 32767.0); +            signal[i * 2 + ch] = static_cast<s16>(std::clamp(out[0][ch], -32768.0, 32767.0));          }      }  } diff --git a/src/core/file_sys/vfs.cpp b/src/core/file_sys/vfs.cpp index a5ec50b1a..b915b4c11 100644 --- a/src/core/file_sys/vfs.cpp +++ b/src/core/file_sys/vfs.cpp @@ -8,6 +8,7 @@  #include "common/common_paths.h"  #include "common/file_util.h"  #include "common/logging/backend.h" +#include "core/file_sys/mode.h"  #include "core/file_sys/vfs.h"  namespace FileSys { diff --git a/src/core/file_sys/vfs.h b/src/core/file_sys/vfs.h index 78a63c59b..22db08b59 100644 --- a/src/core/file_sys/vfs.h +++ b/src/core/file_sys/vfs.h @@ -9,9 +9,8 @@  #include <string_view>  #include <type_traits>  #include <vector> -#include "boost/optional.hpp" +#include <boost/optional.hpp>  #include "common/common_types.h" -#include "core/file_sys/mode.h"  namespace FileSys { @@ -19,6 +18,8 @@ class VfsDirectory;  class VfsFile;  class VfsFilesystem; +enum class Mode : u32; +  // Convenience typedefs to use Vfs* interfaces  using VirtualFilesystem = std::shared_ptr<VfsFilesystem>;  using VirtualDir = std::shared_ptr<VfsDirectory>; diff --git a/src/core/hle/service/am/am.cpp b/src/core/hle/service/am/am.cpp index c524e7a48..78d551a8a 100644 --- a/src/core/hle/service/am/am.cpp +++ b/src/core/hle/service/am/am.cpp @@ -2,6 +2,7 @@  // Licensed under GPLv2 or any later version  // Refer to the license.txt file included. +#include <array>  #include <cinttypes>  #include <stack>  #include "core/core.h" @@ -625,16 +626,16 @@ IApplicationFunctions::IApplicationFunctions() : ServiceFramework("IApplicationF  }  void IApplicationFunctions::PopLaunchParameter(Kernel::HLERequestContext& ctx) { -    constexpr u8 data[0x88] = { +    constexpr std::array<u8, 0x88> data{{          0xca, 0x97, 0x94, 0xc7, // Magic          1,    0,    0,    0,    // IsAccountSelected (bool)          1,    0,    0,    0,    // User Id (word 0)          0,    0,    0,    0,    // User Id (word 1)          0,    0,    0,    0,    // User Id (word 2)          0,    0,    0,    0     // User Id (word 3) -    }; +    }}; -    std::vector<u8> buffer(data, data + sizeof(data)); +    std::vector<u8> buffer(data.begin(), data.end());      IPC::ResponseBuilder rb{ctx, 2, 0, 1}; diff --git a/src/core/hle/service/filesystem/filesystem.cpp b/src/core/hle/service/filesystem/filesystem.cpp index 0d2b1544f..6f9c64263 100644 --- a/src/core/hle/service/filesystem/filesystem.cpp +++ b/src/core/hle/service/filesystem/filesystem.cpp @@ -9,12 +9,12 @@  #include "core/core.h"  #include "core/file_sys/bis_factory.h"  #include "core/file_sys/errors.h" +#include "core/file_sys/mode.h"  #include "core/file_sys/romfs_factory.h"  #include "core/file_sys/savedata_factory.h"  #include "core/file_sys/sdmc_factory.h"  #include "core/file_sys/vfs.h"  #include "core/file_sys/vfs_offset.h" -#include "core/file_sys/vfs_real.h"  #include "core/hle/service/filesystem/filesystem.h"  #include "core/hle/service/filesystem/fsp_ldr.h"  #include "core/hle/service/filesystem/fsp_pr.h" diff --git a/src/core/hle/service/filesystem/filesystem.h b/src/core/hle/service/filesystem/filesystem.h index 572c16f4d..df78be44a 100644 --- a/src/core/hle/service/filesystem/filesystem.h +++ b/src/core/hle/service/filesystem/filesystem.h @@ -7,7 +7,6 @@  #include <memory>  #include "common/common_types.h"  #include "core/file_sys/directory.h" -#include "core/file_sys/mode.h"  #include "core/hle/result.h"  namespace FileSys { @@ -18,6 +17,7 @@ class SaveDataFactory;  class SDMCFactory;  enum class ContentRecordType : u8; +enum class Mode : u32;  enum class SaveDataSpaceId : u8;  enum class StorageId : u8; diff --git a/src/core/hle/service/filesystem/fsp_srv.cpp b/src/core/hle/service/filesystem/fsp_srv.cpp index 8ece74d7e..5759299fe 100644 --- a/src/core/hle/service/filesystem/fsp_srv.cpp +++ b/src/core/hle/service/filesystem/fsp_srv.cpp @@ -15,6 +15,7 @@  #include "common/string_util.h"  #include "core/file_sys/directory.h"  #include "core/file_sys/errors.h" +#include "core/file_sys/mode.h"  #include "core/file_sys/nca_metadata.h"  #include "core/file_sys/savedata_factory.h"  #include "core/file_sys/vfs.h" diff --git a/src/core/hle/service/ns/pl_u.cpp b/src/core/hle/service/ns/pl_u.cpp index bad27894a..53cbf1a6e 100644 --- a/src/core/hle/service/ns/pl_u.cpp +++ b/src/core/hle/service/ns/pl_u.cpp @@ -5,31 +5,98 @@  #include "common/common_paths.h"  #include "common/file_util.h"  #include "core/core.h" +#include "core/file_sys/bis_factory.h" +#include "core/file_sys/romfs.h"  #include "core/hle/ipc_helpers.h" +#include "core/hle/service/filesystem/filesystem.h"  #include "core/hle/service/ns/pl_u.h"  namespace Service::NS { +enum class FontArchives : u64 { +    Extension = 0x0100000000000810, +    Standard = 0x0100000000000811, +    Korean = 0x0100000000000812, +    ChineseTraditional = 0x0100000000000813, +    ChineseSimple = 0x0100000000000814, +}; +  struct FontRegion {      u32 offset;      u32 size;  }; +static constexpr std::array<std::pair<FontArchives, const char*>, 7> SHARED_FONTS{ +    std::make_pair(FontArchives::Standard, "nintendo_udsg-r_std_003.bfttf"), +    std::make_pair(FontArchives::ChineseSimple, "nintendo_udsg-r_org_zh-cn_003.bfttf"), +    std::make_pair(FontArchives::ChineseSimple, "nintendo_udsg-r_ext_zh-cn_003.bfttf"), +    std::make_pair(FontArchives::ChineseTraditional, "nintendo_udjxh-db_zh-tw_003.bfttf"), +    std::make_pair(FontArchives::Korean, "nintendo_udsg-r_ko_003.bfttf"), +    std::make_pair(FontArchives::Extension, "nintendo_ext_003.bfttf"), +    std::make_pair(FontArchives::Extension, "nintendo_ext2_003.bfttf")}; +  // The below data is specific to shared font data dumped from Switch on f/w 2.2  // Virtual address and offsets/sizes likely will vary by dump  static constexpr VAddr SHARED_FONT_MEM_VADDR{0x00000009d3016000ULL}; +static constexpr u32 EXPECTED_RESULT{ +    0x7f9a0218}; // What we expect the decrypted bfttf first 4 bytes to be +static constexpr u32 EXPECTED_MAGIC{ +    0x36f81a1e}; // What we expect the encrypted bfttf first 4 bytes to be  static constexpr u64 SHARED_FONT_MEM_SIZE{0x1100000}; -static constexpr std::array<FontRegion, 6> SHARED_FONT_REGIONS{ -    FontRegion{0x00000008, 0x001fe764}, FontRegion{0x001fe774, 0x00773e58}, -    FontRegion{0x009725d4, 0x0001aca8}, FontRegion{0x0098d284, 0x00369cec}, -    FontRegion{0x00cf6f78, 0x0039b858}, FontRegion{0x010927d8, 0x00019e80}, -}; +static constexpr FontRegion EMPTY_REGION{0, 0}; +std::vector<FontRegion> +    SHARED_FONT_REGIONS{}; // Automatically populated based on shared_fonts dump or system archives + +const FontRegion& GetSharedFontRegion(size_t index) { +    if (index >= SHARED_FONT_REGIONS.size() || SHARED_FONT_REGIONS.empty()) { +        // No font fallback +        return EMPTY_REGION; +    } +    return SHARED_FONT_REGIONS.at(index); +}  enum class LoadState : u32 {      Loading = 0,      Done = 1,  }; +void DecryptSharedFont(const std::vector<u32>& input, std::vector<u8>& output, size_t& offset) { +    ASSERT_MSG(offset + (input.size() * sizeof(u32)) < SHARED_FONT_MEM_SIZE, +               "Shared fonts exceeds 17mb!"); +    ASSERT_MSG(input[0] == EXPECTED_MAGIC, "Failed to derive key, unexpected magic number"); + +    const u32 KEY = input[0] ^ EXPECTED_RESULT; // Derive key using an inverse xor +    std::vector<u32> transformed_font(input.size()); +    // TODO(ogniK): Figure out a better way to do this +    std::transform(input.begin(), input.end(), transformed_font.begin(), +                   [&KEY](u32 font_data) { return Common::swap32(font_data ^ KEY); }); +    transformed_font[1] = Common::swap32(transformed_font[1]) ^ KEY; // "re-encrypt" the size +    std::memcpy(output.data() + offset, transformed_font.data(), +                transformed_font.size() * sizeof(u32)); +    offset += transformed_font.size() * sizeof(u32); +} + +static u32 GetU32Swapped(const u8* data) { +    u32 value; +    std::memcpy(&value, data, sizeof(value)); +    return Common::swap32(value); // Helper function to make BuildSharedFontsRawRegions a bit nicer +} + +void BuildSharedFontsRawRegions(const std::vector<u8>& input) { +    unsigned cur_offset = 0; // As we can derive the xor key we can just populate the offsets based +                             // on the shared memory dump +    for (size_t i = 0; i < SHARED_FONTS.size(); i++) { +        // Out of shared fonts/Invalid font +        if (GetU32Swapped(input.data() + cur_offset) != EXPECTED_RESULT) +            break; +        const u32 KEY = GetU32Swapped(input.data() + cur_offset) ^ +                        EXPECTED_MAGIC; // Derive key withing inverse xor +        const u32 SIZE = GetU32Swapped(input.data() + cur_offset + 4) ^ KEY; +        SHARED_FONT_REGIONS.push_back(FontRegion{cur_offset + 8, SIZE}); +        cur_offset += SIZE + 8; +    } +} +  PL_U::PL_U() : ServiceFramework("pl:u") {      static const FunctionInfo functions[] = {          {0, &PL_U::RequestLoad, "RequestLoad"}, @@ -40,26 +107,78 @@ PL_U::PL_U() : ServiceFramework("pl:u") {          {5, &PL_U::GetSharedFontInOrderOfPriority, "GetSharedFontInOrderOfPriority"},      };      RegisterHandlers(functions); -      // Attempt to load shared font data from disk -    const std::string filepath{FileUtil::GetUserPath(FileUtil::UserPath::SysDataDir) + SHARED_FONT}; -    FileUtil::CreateFullPath(filepath); // Create path if not already created -    FileUtil::IOFile file(filepath, "rb"); - -    shared_font = std::make_shared<std::vector<u8>>(SHARED_FONT_MEM_SIZE); -    if (file.IsOpen()) { -        // Read shared font data -        ASSERT(file.GetSize() == SHARED_FONT_MEM_SIZE); -        file.ReadBytes(shared_font->data(), shared_font->size()); +    const auto nand = FileSystem::GetSystemNANDContents(); +    // Rebuild shared fonts from data ncas +    if (nand->HasEntry(static_cast<u64>(FontArchives::Standard), +                       FileSys::ContentRecordType::Data)) { +        size_t offset = 0; +        shared_font = std::make_shared<std::vector<u8>>(SHARED_FONT_MEM_SIZE); +        for (auto font : SHARED_FONTS) { +            const auto nca = +                nand->GetEntry(static_cast<u64>(font.first), FileSys::ContentRecordType::Data); +            if (!nca) { +                LOG_ERROR(Service_NS, "Failed to find {:016X}! Skipping", +                          static_cast<u64>(font.first)); +                continue; +            } +            const auto romfs = nca->GetRomFS(); +            if (!romfs) { +                LOG_ERROR(Service_NS, "{:016X} has no RomFS! Skipping", +                          static_cast<u64>(font.first)); +                continue; +            } +            const auto extracted_romfs = FileSys::ExtractRomFS(romfs); +            if (!extracted_romfs) { +                LOG_ERROR(Service_NS, "Failed to extract RomFS for {:016X}! Skipping", +                          static_cast<u64>(font.first)); +                continue; +            } +            const auto font_fp = extracted_romfs->GetFile(font.second); +            if (!font_fp) { +                LOG_ERROR(Service_NS, "{:016X} has no file \"{}\"! Skipping", +                          static_cast<u64>(font.first), font.second); +                continue; +            } +            std::vector<u32> font_data_u32(font_fp->GetSize() / sizeof(u32)); +            font_fp->ReadBytes<u32>(font_data_u32.data(), font_fp->GetSize()); +            // We need to be BigEndian as u32s for the xor encryption +            std::transform(font_data_u32.begin(), font_data_u32.end(), font_data_u32.begin(), +                           Common::swap32); +            FontRegion region{ +                static_cast<u32>(offset + 8), +                static_cast<u32>((font_data_u32.size() * sizeof(u32)) - +                                 8)}; // Font offset and size do not account for the header +            DecryptSharedFont(font_data_u32, *shared_font, offset); +            SHARED_FONT_REGIONS.push_back(region); +        }      } else { -        LOG_WARNING(Service_NS, "Unable to load shared font: {}", filepath); +        const std::string filepath{FileUtil::GetUserPath(FileUtil::UserPath::SysDataDir) + +                                   SHARED_FONT}; +        // Create path if not already created +        if (!FileUtil::CreateFullPath(filepath)) { +            LOG_ERROR(Service_NS, "Failed to create sharedfonts path \"{}\"!", filepath); +            return; +        } +        FileUtil::IOFile file(filepath, "rb"); + +        shared_font = std::make_shared<std::vector<u8>>( +            SHARED_FONT_MEM_SIZE); // Shared memory needs to always be allocated and a fixed size +        if (file.IsOpen()) { +            // Read shared font data +            ASSERT(file.GetSize() == SHARED_FONT_MEM_SIZE); +            file.ReadBytes(shared_font->data(), shared_font->size()); +            BuildSharedFontsRawRegions(*shared_font); +        } else { +            LOG_WARNING(Service_NS, "Unable to load shared font: {}", filepath); +        }      }  }  void PL_U::RequestLoad(Kernel::HLERequestContext& ctx) {      IPC::RequestParser rp{ctx};      const u32 shared_font_type{rp.Pop<u32>()}; - +    // Games don't call this so all fonts should be loaded      LOG_DEBUG(Service_NS, "called, shared_font_type={}", shared_font_type);      IPC::ResponseBuilder rb{ctx, 2};      rb.Push(RESULT_SUCCESS); @@ -82,7 +201,7 @@ void PL_U::GetSize(Kernel::HLERequestContext& ctx) {      LOG_DEBUG(Service_NS, "called, font_id={}", font_id);      IPC::ResponseBuilder rb{ctx, 3};      rb.Push(RESULT_SUCCESS); -    rb.Push<u32>(SHARED_FONT_REGIONS[font_id].size); +    rb.Push<u32>(GetSharedFontRegion(font_id).size);  }  void PL_U::GetSharedMemoryAddressOffset(Kernel::HLERequestContext& ctx) { @@ -92,14 +211,10 @@ void PL_U::GetSharedMemoryAddressOffset(Kernel::HLERequestContext& ctx) {      LOG_DEBUG(Service_NS, "called, font_id={}", font_id);      IPC::ResponseBuilder rb{ctx, 3};      rb.Push(RESULT_SUCCESS); -    rb.Push<u32>(SHARED_FONT_REGIONS[font_id].offset); +    rb.Push<u32>(GetSharedFontRegion(font_id).offset);  }  void PL_U::GetSharedMemoryNativeHandle(Kernel::HLERequestContext& ctx) { -    // TODO(bunnei): This is a less-than-ideal solution to load a RAM dump of the Switch shared -    // font data. This (likely) relies on exact address, size, and offsets from the original -    // dump. In the future, we need to replace this with a more robust solution. -      // Map backing memory for the font data      Core::CurrentProcess()->vm_manager.MapMemoryBlock(          SHARED_FONT_MEM_VADDR, shared_font, 0, SHARED_FONT_MEM_SIZE, Kernel::MemoryState::Shared); @@ -128,8 +243,9 @@ void PL_U::GetSharedFontInOrderOfPriority(Kernel::HLERequestContext& ctx) {      // TODO(ogniK): Have actual priority order      for (size_t i = 0; i < SHARED_FONT_REGIONS.size(); i++) {          font_codes.push_back(static_cast<u32>(i)); -        font_offsets.push_back(SHARED_FONT_REGIONS[i].offset); -        font_sizes.push_back(SHARED_FONT_REGIONS[i].size); +        auto region = GetSharedFontRegion(i); +        font_offsets.push_back(region.offset); +        font_sizes.push_back(region.size);      }      ctx.WriteBuffer(font_codes, 0); diff --git a/src/video_core/engines/shader_bytecode.h b/src/video_core/engines/shader_bytecode.h index 875b90359..67194b0e3 100644 --- a/src/video_core/engines/shader_bytecode.h +++ b/src/video_core/engines/shader_bytecode.h @@ -518,7 +518,7 @@ union Instruction {                  return TextureType::Texture1D;              }              if (texture_info == 2 || texture_info == 8 || texture_info == 12 || -                texture_info >= 4 && texture_info <= 6) { +                (texture_info >= 4 && texture_info <= 6)) {                  return TextureType::Texture2D;              }              if (texture_info == 7) { diff --git a/src/video_core/renderer_opengl/gl_shader_decompiler.cpp b/src/video_core/renderer_opengl/gl_shader_decompiler.cpp index aeb908744..5b976b636 100644 --- a/src/video_core/renderer_opengl/gl_shader_decompiler.cpp +++ b/src/video_core/renderer_opengl/gl_shader_decompiler.cpp @@ -440,12 +440,13 @@ public:          }          declarations.AddNewLine(); -        const auto& samplers = GetSamplers(); -        for (const auto& sampler : samplers) { -            declarations.AddLine("uniform " + sampler.GetTypeString() + ' ' + sampler.GetName() + -                                 ';'); +        // Append the sampler2D array for the used textures. +        size_t num_samplers = GetSamplers().size(); +        if (num_samplers > 0) { +            declarations.AddLine("uniform sampler2D " + SamplerEntry::GetArrayName(stage) + '[' + +                                 std::to_string(num_samplers) + "];"); +            declarations.AddNewLine();          } -        declarations.AddNewLine();      }      /// Returns a list of constant buffer declarations @@ -457,14 +458,13 @@ public:      }      /// Returns a list of samplers used in the shader -    const std::vector<SamplerEntry>& GetSamplers() const { +    std::vector<SamplerEntry> GetSamplers() const {          return used_samplers;      }      /// Returns the GLSL sampler used for the input shader sampler, and creates a new one if      /// necessary. -    std::string AccessSampler(const Sampler& sampler, Tegra::Shader::TextureType type, -                              bool is_array) { +    std::string AccessSampler(const Sampler& sampler) {          size_t offset = static_cast<size_t>(sampler.index.Value());          // If this sampler has already been used, return the existing mapping. @@ -473,13 +473,12 @@ public:                           [&](const SamplerEntry& entry) { return entry.GetOffset() == offset; });          if (itr != used_samplers.end()) { -            ASSERT(itr->GetType() == type && itr->IsArray() == is_array);              return itr->GetName();          }          // Otherwise create a new mapping for this sampler          size_t next_index = used_samplers.size(); -        SamplerEntry entry{stage, offset, next_index, type, is_array}; +        SamplerEntry entry{stage, offset, next_index};          used_samplers.emplace_back(entry);          return entry.GetName();      } @@ -657,8 +656,8 @@ private:      }      /// Generates code representing a texture sampler. -    std::string GetSampler(const Sampler& sampler, Tegra::Shader::TextureType type, bool is_array) { -        return regs.AccessSampler(sampler, type, is_array); +    std::string GetSampler(const Sampler& sampler) { +        return regs.AccessSampler(sampler);      }      /** @@ -1556,39 +1555,10 @@ private:                  break;              }              case OpCode::Id::TEX: { -                ASSERT_MSG(instr.tex.array == 0, "TEX arrays unimplemented"); -                std::string coord{}; - -                switch (instr.tex.texture_type) { -                case Tegra::Shader::TextureType::Texture2D: { -                    std::string x = regs.GetRegisterAsFloat(instr.gpr8); -                    std::string y = regs.GetRegisterAsFloat(instr.gpr8.Value() + 1); -                    coord = "vec2 coords = vec2(" + x + ", " + y + ");"; -                    break; -                } -                case Tegra::Shader::TextureType::Texture3D: { -                    std::string x = regs.GetRegisterAsFloat(instr.gpr8); -                    std::string y = regs.GetRegisterAsFloat(instr.gpr8.Value() + 1); -                    std::string z = regs.GetRegisterAsFloat(instr.gpr20); -                    coord = "vec3 coords = vec3(" + x + ", " + y + ", " + z + ");"; -                    break; -                } -                case Tegra::Shader::TextureType::TextureCube: { -                    std::string x = regs.GetRegisterAsFloat(instr.gpr8); -                    std::string y = regs.GetRegisterAsFloat(instr.gpr8.Value() + 1); -                    std::string z = regs.GetRegisterAsFloat(instr.gpr8.Value() + 2); -                    ASSERT(instr.gpr20.Value() == Register::ZeroIndex); -                    coord = "vec3 coords = vec3(" + x + ", " + y + ", " + z + ");"; -                    break; -                } -                default: -                    LOG_CRITICAL(HW_GPU, "Unhandled texture type {}", -                                 static_cast<u32>(instr.tex.texture_type.Value())); -                    UNREACHABLE(); -                } - -                const std::string sampler = -                    GetSampler(instr.sampler, instr.tex.texture_type, instr.tex.array); +                const std::string op_a = regs.GetRegisterAsFloat(instr.gpr8); +                const std::string op_b = regs.GetRegisterAsFloat(instr.gpr8.Value() + 1); +                const std::string sampler = GetSampler(instr.sampler); +                const std::string coord = "vec2 coords = vec2(" + op_a + ", " + op_b + ");";                  // Add an extra scope and declare the texture coords inside to prevent                  // overwriting them in case they are used as outputs of the texs instruction.                  shader.AddLine("{"); @@ -1610,72 +1580,20 @@ private:                  break;              }              case OpCode::Id::TEXS: { -                std::string coord{}; - -                switch (instr.texs.GetTextureType()) { -                case Tegra::Shader::TextureType::Texture2D: { -                    if (instr.texs.IsArrayTexture()) { -                        std::string index = regs.GetRegisterAsInteger(instr.gpr8); -                        std::string x = regs.GetRegisterAsFloat(instr.gpr8.Value() + 1); -                        std::string y = regs.GetRegisterAsFloat(instr.gpr20); -                        coord = "vec3 coords = vec3(" + x + ", " + y + ", " + index + ");"; -                    } else { -                        std::string x = regs.GetRegisterAsFloat(instr.gpr8); -                        std::string y = regs.GetRegisterAsFloat(instr.gpr20); -                        coord = "vec2 coords = vec2(" + x + ", " + y + ");"; -                    } -                    break; -                } -                case Tegra::Shader::TextureType::Texture3D: { -                    std::string x = regs.GetRegisterAsFloat(instr.gpr8); -                    std::string y = regs.GetRegisterAsFloat(instr.gpr20); -                    std::string z = regs.GetRegisterAsFloat(instr.gpr20.Value() + 1); -                    coord = "vec3 coords = vec3(" + x + ", " + y + ", " + z + ");"; -                    break; -                } -                case Tegra::Shader::TextureType::TextureCube: { -                    std::string x = regs.GetRegisterAsFloat(instr.gpr8); -                    std::string y = regs.GetRegisterAsFloat(instr.gpr8.Value() + 1); -                    std::string z = regs.GetRegisterAsFloat(instr.gpr20); -                    coord = "vec3 coords = vec3(" + x + ", " + y + ", " + z + ");"; -                    break; -                } -                default: -                    LOG_CRITICAL(HW_GPU, "Unhandled texture type {}", -                                 static_cast<u32>(instr.texs.GetTextureType())); -                    UNREACHABLE(); -                } -                const std::string sampler = GetSampler(instr.sampler, instr.texs.GetTextureType(), -                                                       instr.texs.IsArrayTexture()); +                const std::string op_a = regs.GetRegisterAsFloat(instr.gpr8); +                const std::string op_b = regs.GetRegisterAsFloat(instr.gpr20); +                const std::string sampler = GetSampler(instr.sampler); +                const std::string coord = "vec2 coords = vec2(" + op_a + ", " + op_b + ");";                  const std::string texture = "texture(" + sampler + ", coords)";                  WriteTexsInstruction(instr, coord, texture);                  break;              }              case OpCode::Id::TLDS: { -                ASSERT(instr.tlds.GetTextureType() == Tegra::Shader::TextureType::Texture2D); -                ASSERT(instr.tlds.IsArrayTexture() == false); -                std::string coord{}; - -                switch (instr.tlds.GetTextureType()) { -                case Tegra::Shader::TextureType::Texture2D: { -                    if (instr.tlds.IsArrayTexture()) { -                        LOG_CRITICAL(HW_GPU, "Unhandled 2d array texture"); -                        UNREACHABLE(); -                    } else { -                        std::string x = regs.GetRegisterAsInteger(instr.gpr8); -                        std::string y = regs.GetRegisterAsInteger(instr.gpr20); -                        coord = "ivec2 coords = ivec2(" + x + ", " + y + ");"; -                    } -                    break; -                } -                default: -                    LOG_CRITICAL(HW_GPU, "Unhandled texture type {}", -                                 static_cast<u32>(instr.tlds.GetTextureType())); -                    UNREACHABLE(); -                } -                const std::string sampler = GetSampler(instr.sampler, instr.tlds.GetTextureType(), -                                                       instr.tlds.IsArrayTexture()); +                const std::string op_a = regs.GetRegisterAsInteger(instr.gpr8); +                const std::string op_b = regs.GetRegisterAsInteger(instr.gpr20); +                const std::string sampler = GetSampler(instr.sampler); +                const std::string coord = "ivec2 coords = ivec2(" + op_a + ", " + op_b + ");";                  const std::string texture = "texelFetch(" + sampler + ", coords, 0)";                  WriteTexsInstruction(instr, coord, texture);                  break; @@ -1698,8 +1616,7 @@ private:                      UNREACHABLE();                  } -                const std::string sampler = -                    GetSampler(instr.sampler, instr.tld4.texture_type, instr.tld4.array); +                const std::string sampler = GetSampler(instr.sampler);                  // Add an extra scope and declare the texture coords inside to prevent                  // overwriting them in case they are used as outputs of the texs instruction.                  shader.AddLine("{"); @@ -1725,8 +1642,7 @@ private:                  const std::string op_a = regs.GetRegisterAsFloat(instr.gpr8);                  const std::string op_b = regs.GetRegisterAsFloat(instr.gpr20);                  // TODO(Subv): Figure out how the sampler type is encoded in the TLD4S instruction. -                const std::string sampler = -                    GetSampler(instr.sampler, Tegra::Shader::TextureType::Texture2D, false); +                const std::string sampler = GetSampler(instr.sampler);                  const std::string coord = "vec2 coords = vec2(" + op_a + ", " + op_b + ");";                  const std::string texture = "textureGather(" + sampler + ", coords, " +                                              std::to_string(instr.tld4s.component) + ')'; diff --git a/src/video_core/renderer_opengl/gl_shader_gen.h b/src/video_core/renderer_opengl/gl_shader_gen.h index db48da645..4729ce0fc 100644 --- a/src/video_core/renderer_opengl/gl_shader_gen.h +++ b/src/video_core/renderer_opengl/gl_shader_gen.h @@ -11,7 +11,6 @@  #include <vector>  #include "common/common_types.h"  #include "common/hash.h" -#include "video_core/engines/shader_bytecode.h"  namespace GLShader { @@ -73,9 +72,8 @@ class SamplerEntry {      using Maxwell = Tegra::Engines::Maxwell3D::Regs;  public: -    SamplerEntry(Maxwell::ShaderStage stage, size_t offset, size_t index, -                 Tegra::Shader::TextureType type, bool is_array) -        : offset(offset), stage(stage), sampler_index(index), type(type), is_array(is_array) {} +    SamplerEntry(Maxwell::ShaderStage stage, size_t offset, size_t index) +        : offset(offset), stage(stage), sampler_index(index) {}      size_t GetOffset() const {          return offset; @@ -90,41 +88,8 @@ public:      }      std::string GetName() const { -        return std::string(TextureSamplerNames[static_cast<size_t>(stage)]) + '_' + -               std::to_string(sampler_index); -    } - -    std::string GetTypeString() const { -        using Tegra::Shader::TextureType; -        std::string glsl_type; - -        switch (type) { -        case TextureType::Texture1D: -            glsl_type = "sampler1D"; -            break; -        case TextureType::Texture2D: -            glsl_type = "sampler2D"; -            break; -        case TextureType::Texture3D: -            glsl_type = "sampler3D"; -            break; -        case TextureType::TextureCube: -            glsl_type = "samplerCube"; -            break; -        default: -            UNIMPLEMENTED(); -        } -        if (is_array) -            glsl_type += "Array"; -        return glsl_type; -    } - -    Tegra::Shader::TextureType GetType() const { -        return type; -    } - -    bool IsArray() const { -        return is_array; +        return std::string(TextureSamplerNames[static_cast<size_t>(stage)]) + '[' + +               std::to_string(sampler_index) + ']';      }      static std::string GetArrayName(Maxwell::ShaderStage stage) { @@ -135,14 +100,11 @@ private:      static constexpr std::array<const char*, Maxwell::MaxShaderStage> TextureSamplerNames = {          "tex_vs", "tex_tessc", "tex_tesse", "tex_gs", "tex_fs",      }; -      /// Offset in TSC memory from which to read the sampler object, as specified by the sampling      /// instruction.      size_t offset; -    Maxwell::ShaderStage stage;      ///< Shader stage where this sampler was used. -    size_t sampler_index;            ///< Value used to index into the generated GLSL sampler array. -    Tegra::Shader::TextureType type; ///< The type used to sample this texture (Texture2D, etc) -    bool is_array; ///< Whether the texture is being sampled as an array texture or not. +    Maxwell::ShaderStage stage; ///< Shader stage where this sampler was used. +    size_t sampler_index;       ///< Value used to index into the generated GLSL sampler array.  };  struct ShaderEntries { | 
