diff options
| -rw-r--r-- | appveyor.yml | 2 | ||||
| -rw-r--r-- | src/core/arm/unicorn/arm_unicorn.cpp | 38 | ||||
| -rw-r--r-- | src/core/arm/unicorn/arm_unicorn.h | 4 | ||||
| -rw-r--r-- | src/core/gdbstub/gdbstub.cpp | 170 | ||||
| -rw-r--r-- | src/core/gdbstub/gdbstub.h | 9 | ||||
| -rw-r--r-- | src/video_core/engines/shader_bytecode.h | 63 | ||||
| -rw-r--r-- | src/video_core/renderer_opengl/gl_shader_decompiler.cpp | 80 | 
7 files changed, 312 insertions, 54 deletions
diff --git a/appveyor.yml b/appveyor.yml index 4f928adb5..72cda26a7 100644 --- a/appveyor.yml +++ b/appveyor.yml @@ -53,7 +53,7 @@ build_script:            # https://www.appveyor.com/docs/build-phase            msbuild msvc_build/yuzu.sln /maxcpucount /logger:"C:\Program Files\AppVeyor\BuildAgent\Appveyor.MSBuildLogger.dll"          } else { -          C:\msys64\usr\bin\bash.exe -lc 'mingw32-make -j4 -C mingw_build/ 2>&1' +          C:\msys64\usr\bin\bash.exe -lc 'mingw32-make -C mingw_build/ 2>&1'          }  after_build: diff --git a/src/core/arm/unicorn/arm_unicorn.cpp b/src/core/arm/unicorn/arm_unicorn.cpp index c0cc62f03..ce6c5616d 100644 --- a/src/core/arm/unicorn/arm_unicorn.cpp +++ b/src/core/arm/unicorn/arm_unicorn.cpp @@ -35,6 +35,17 @@ LoadDll LoadDll::g_load_dll;          }                                                                                          \      } while (0) +static void CodeHook(uc_engine* uc, uint64_t address, uint32_t size, void* user_data) { +    GDBStub::BreakpointAddress bkpt = +        GDBStub::GetNextBreakpointFromAddress(address, GDBStub::BreakpointType::Execute); +    if (GDBStub::IsMemoryBreak() || +        (bkpt.type != GDBStub::BreakpointType::None && address == bkpt.address)) { +        auto core = static_cast<ARM_Unicorn*>(user_data); +        core->RecordBreak(bkpt); +        uc_emu_stop(uc); +    } +} +  static void InterruptHook(uc_engine* uc, u32 intNo, void* user_data) {      u32 esr{};      CHECKED(uc_reg_read(uc, UC_ARM64_REG_ESR, &esr)); @@ -67,6 +78,10 @@ ARM_Unicorn::ARM_Unicorn() {      uc_hook hook{};      CHECKED(uc_hook_add(uc, &hook, UC_HOOK_INTR, (void*)InterruptHook, this, 0, -1));      CHECKED(uc_hook_add(uc, &hook, UC_HOOK_MEM_INVALID, (void*)UnmappedMemoryHook, this, 0, -1)); +    if (GDBStub::IsServerEnabled()) { +        CHECKED(uc_hook_add(uc, &hook, UC_HOOK_CODE, (void*)CodeHook, this, 0, -1)); +        last_bkpt_hit = false; +    }  }  ARM_Unicorn::~ARM_Unicorn() { @@ -155,7 +170,11 @@ void ARM_Unicorn::SetTlsAddress(VAddr base) {  }  void ARM_Unicorn::Run() { -    ExecuteInstructions(std::max(CoreTiming::GetDowncount(), 0)); +    if (GDBStub::IsServerEnabled()) { +        ExecuteInstructions(std::max(4000000, 0)); +    } else { +        ExecuteInstructions(std::max(CoreTiming::GetDowncount(), 0)); +    }  }  void ARM_Unicorn::Step() { @@ -168,6 +187,18 @@ void ARM_Unicorn::ExecuteInstructions(int num_instructions) {      MICROPROFILE_SCOPE(ARM_Jit);      CHECKED(uc_emu_start(uc, GetPC(), 1ULL << 63, 0, num_instructions));      CoreTiming::AddTicks(num_instructions); +    if (GDBStub::IsServerEnabled()) { +        if (last_bkpt_hit) { +            uc_reg_write(uc, UC_ARM64_REG_PC, &last_bkpt.address); +        } +        Kernel::Thread* thread = Kernel::GetCurrentThread(); +        SaveContext(thread->context); +        if (last_bkpt_hit) { +            last_bkpt_hit = false; +            GDBStub::Break(); +        } +        GDBStub::SendTrap(thread, 5); +    }  }  void ARM_Unicorn::SaveContext(ARM_Interface::ThreadContext& ctx) { @@ -233,3 +264,8 @@ void ARM_Unicorn::PrepareReschedule() {  }  void ARM_Unicorn::ClearInstructionCache() {} + +void ARM_Unicorn::RecordBreak(GDBStub::BreakpointAddress bkpt) { +    last_bkpt = bkpt; +    last_bkpt_hit = true; +} diff --git a/src/core/arm/unicorn/arm_unicorn.h b/src/core/arm/unicorn/arm_unicorn.h index b99b58e4c..a482a2aa3 100644 --- a/src/core/arm/unicorn/arm_unicorn.h +++ b/src/core/arm/unicorn/arm_unicorn.h @@ -7,6 +7,7 @@  #include <unicorn/unicorn.h>  #include "common/common_types.h"  #include "core/arm/arm_interface.h" +#include "core/gdbstub/gdbstub.h"  class ARM_Unicorn final : public ARM_Interface {  public: @@ -35,7 +36,10 @@ public:      void Step() override;      void ClearInstructionCache() override;      void PageTableChanged() override{}; +    void RecordBreak(GDBStub::BreakpointAddress bkpt);  private:      uc_engine* uc{}; +    GDBStub::BreakpointAddress last_bkpt{}; +    bool last_bkpt_hit;  }; diff --git a/src/core/gdbstub/gdbstub.cpp b/src/core/gdbstub/gdbstub.cpp index 6c5a40ba8..2603192fe 100644 --- a/src/core/gdbstub/gdbstub.cpp +++ b/src/core/gdbstub/gdbstub.cpp @@ -32,9 +32,13 @@  #include "common/logging/log.h"  #include "common/string_util.h" +#include "common/swap.h"  #include "core/arm/arm_interface.h"  #include "core/core.h" +#include "core/core_cpu.h"  #include "core/gdbstub/gdbstub.h" +#include "core/hle/kernel/kernel.h" +#include "core/hle/kernel/scheduler.h"  #include "core/loader/loader.h"  #include "core/memory.h" @@ -137,15 +141,17 @@ static u8 command_buffer[GDB_BUFFER_SIZE];  static u32 command_length;  static u32 latest_signal = 0; -static bool step_break = false;  static bool memory_break = false; +static Kernel::Thread* current_thread = nullptr; +  // Binding to a port within the reserved ports range (0-1023) requires root permissions,  // so default to a port outside of that range.  static u16 gdbstub_port = 24689;  static bool halt_loop = true;  static bool step_loop = false; +static bool send_trap = false;  // If set to false, the server will never be started and no  // gdbstub-related functions will be executed. @@ -165,6 +171,53 @@ static std::map<u64, Breakpoint> breakpoints_execute;  static std::map<u64, Breakpoint> breakpoints_read;  static std::map<u64, Breakpoint> breakpoints_write; +static Kernel::Thread* FindThreadById(int id) { +    for (int core = 0; core < Core::NUM_CPU_CORES; core++) { +        auto threads = Core::System::GetInstance().Scheduler(core)->GetThreadList(); +        for (auto thread : threads) { +            if (thread->GetThreadId() == id) { +                current_thread = thread.get(); +                return current_thread; +            } +        } +    } +    return nullptr; +} + +static u64 RegRead(int id, Kernel::Thread* thread = nullptr) { +    if (!thread) { +        return 0; +    } + +    if (id < SP_REGISTER) { +        return thread->context.cpu_registers[id]; +    } else if (id == SP_REGISTER) { +        return thread->context.sp; +    } else if (id == PC_REGISTER) { +        return thread->context.pc; +    } else if (id == CPSR_REGISTER) { +        return thread->context.cpsr; +    } else { +        return 0; +    } +} + +static void RegWrite(int id, u64 val, Kernel::Thread* thread = nullptr) { +    if (!thread) { +        return; +    } + +    if (id < SP_REGISTER) { +        thread->context.cpu_registers[id] = val; +    } else if (id == SP_REGISTER) { +        thread->context.sp = val; +    } else if (id == PC_REGISTER) { +        thread->context.pc = val; +    } else if (id == CPSR_REGISTER) { +        thread->context.cpsr = val; +    } +} +  /**   * Turns hex string character into the equivalent byte.   * @@ -193,7 +246,7 @@ static u8 NibbleToHex(u8 n) {      if (n < 0xA) {          return '0' + n;      } else { -        return 'A' + n - 0xA; +        return 'a' + n - 0xA;      }  } @@ -439,6 +492,8 @@ static void SendReply(const char* reply) {          return;      } +    NGLOG_DEBUG(Debug_GDBStub, "Reply: {}", reply); +      memset(command_buffer, 0, sizeof(command_buffer));      command_length = static_cast<u32>(strlen(reply)); @@ -483,6 +538,22 @@ static void HandleQuery() {      } else if (strncmp(query, "Xfer:features:read:target.xml:",                         strlen("Xfer:features:read:target.xml:")) == 0) {          SendReply(target_xml); +    } else if (strncmp(query, "Offsets", strlen("Offsets")) == 0) { +        std::string buffer = fmt::format("TextSeg={:0x}", Memory::PROCESS_IMAGE_VADDR); +        SendReply(buffer.c_str()); +    } else if (strncmp(query, "fThreadInfo", strlen("fThreadInfo")) == 0) { +        std::string val = "m"; +        for (int core = 0; core < Core::NUM_CPU_CORES; core++) { +            auto threads = Core::System::GetInstance().Scheduler(core)->GetThreadList(); +            for (auto thread : threads) { +                val += fmt::format("{:x}", thread->GetThreadId()); +                val += ","; +            } +        } +        val.pop_back(); +        SendReply(val.c_str()); +    } else if (strncmp(query, "sThreadInfo", strlen("sThreadInfo")) == 0) { +        SendReply("l");      } else {          SendReply("");      } @@ -490,11 +561,40 @@ static void HandleQuery() {  /// Handle set thread command from gdb client.  static void HandleSetThread() { -    if (memcmp(command_buffer, "Hg0", 3) == 0 || memcmp(command_buffer, "Hc-1", 4) == 0 || -        memcmp(command_buffer, "Hc0", 4) == 0 || memcmp(command_buffer, "Hc1", 4) == 0) { -        return SendReply("OK"); +    if (memcmp(command_buffer, "Hc", 2) == 0 || memcmp(command_buffer, "Hg", 2) == 0) { +        int thread_id = -1; +        if (command_buffer[2] != '-') { +            thread_id = static_cast<int>(HexToInt( +                command_buffer + 2, +                command_length - 2 /*strlen(reinterpret_cast<char*>(command_buffer) + 2)*/)); +        } +        if (thread_id >= 1) { +            current_thread = FindThreadById(thread_id); +        } +        if (!current_thread) { +            thread_id = 1; +            current_thread = FindThreadById(thread_id); +        } +        if (current_thread) { +            SendReply("OK"); +            return; +        }      } +    SendReply("E01"); +} +/// Handle thread alive command from gdb client. +static void HandleThreadAlive() { +    int thread_id = static_cast<int>( +        HexToInt(command_buffer + 1, +                 command_length - 1 /*strlen(reinterpret_cast<char*>(command_buffer) + 1)*/)); +    if (thread_id == 0) { +        thread_id = 1; +    } +    if (FindThreadById(thread_id)) { +        SendReply("OK"); +        return; +    }      SendReply("E01");  } @@ -503,15 +603,24 @@ static void HandleSetThread() {   *   * @param signal Signal to be sent to client.   */ -static void SendSignal(u32 signal) { +static void SendSignal(Kernel::Thread* thread, u32 signal, bool full = true) {      if (gdbserver_socket == -1) {          return;      }      latest_signal = signal; -    std::string buffer = fmt::format("T{:02x}", latest_signal); -    NGLOG_DEBUG(Debug_GDBStub, "Response: {}", buffer); +    std::string buffer; +    if (full) { +        buffer = fmt::format("T{:02x}{:02x}:{:016x};{:02x}:{:016x};", latest_signal, PC_REGISTER, +                             Common::swap64(RegRead(PC_REGISTER, thread)), SP_REGISTER, +                             Common::swap64(RegRead(SP_REGISTER, thread))); +    } else { +        buffer = fmt::format("T{:02x};", latest_signal); +    } + +    buffer += fmt::format("thread:{:x};", thread->GetThreadId()); +      SendReply(buffer.c_str());  } @@ -527,7 +636,7 @@ static void ReadCommand() {      } else if (c == 0x03) {          NGLOG_INFO(Debug_GDBStub, "gdb: found break command");          halt_loop = true; -        SendSignal(SIGTRAP); +        SendSignal(current_thread, SIGTRAP);          return;      } else if (c != GDB_STUB_START) {          NGLOG_DEBUG(Debug_GDBStub, "gdb: read invalid byte {:02X}", c); @@ -598,11 +707,11 @@ static void ReadRegister() {      }      if (id <= SP_REGISTER) { -        LongToGdbHex(reply, Core::CurrentArmInterface().GetReg(static_cast<int>(id))); +        LongToGdbHex(reply, RegRead(id, current_thread));      } else if (id == PC_REGISTER) { -        LongToGdbHex(reply, Core::CurrentArmInterface().GetPC()); +        LongToGdbHex(reply, RegRead(id, current_thread));      } else if (id == CPSR_REGISTER) { -        IntToGdbHex(reply, Core::CurrentArmInterface().GetCPSR()); +        IntToGdbHex(reply, (u32)RegRead(id, current_thread));      } else {          return SendReply("E01");      } @@ -618,16 +727,16 @@ static void ReadRegisters() {      u8* bufptr = buffer;      for (int reg = 0; reg <= SP_REGISTER; reg++) { -        LongToGdbHex(bufptr + reg * 16, Core::CurrentArmInterface().GetReg(reg)); +        LongToGdbHex(bufptr + reg * 16, RegRead(reg, current_thread));      }      bufptr += (32 * 16); -    LongToGdbHex(bufptr, Core::CurrentArmInterface().GetPC()); +    LongToGdbHex(bufptr, RegRead(PC_REGISTER, current_thread));      bufptr += 16; -    IntToGdbHex(bufptr, Core::CurrentArmInterface().GetCPSR()); +    IntToGdbHex(bufptr, (u32)RegRead(CPSR_REGISTER, current_thread));      bufptr += 8; @@ -646,11 +755,11 @@ static void WriteRegister() {      }      if (id <= SP_REGISTER) { -        Core::CurrentArmInterface().SetReg(id, GdbHexToLong(buffer_ptr)); +        RegWrite(id, GdbHexToLong(buffer_ptr), current_thread);      } else if (id == PC_REGISTER) { -        Core::CurrentArmInterface().SetPC(GdbHexToLong(buffer_ptr)); +        RegWrite(id, GdbHexToLong(buffer_ptr), current_thread);      } else if (id == CPSR_REGISTER) { -        Core::CurrentArmInterface().SetCPSR(GdbHexToInt(buffer_ptr)); +        RegWrite(id, GdbHexToInt(buffer_ptr), current_thread);      } else {          return SendReply("E01");      } @@ -667,11 +776,11 @@ static void WriteRegisters() {      for (int i = 0, reg = 0; reg <= CPSR_REGISTER; i++, reg++) {          if (reg <= SP_REGISTER) { -            Core::CurrentArmInterface().SetReg(reg, GdbHexToLong(buffer_ptr + i * 16)); +            RegWrite(reg, GdbHexToLong(buffer_ptr + i * 16), current_thread);          } else if (reg == PC_REGISTER) { -            Core::CurrentArmInterface().SetPC(GdbHexToLong(buffer_ptr + i * 16)); +            RegWrite(PC_REGISTER, GdbHexToLong(buffer_ptr + i * 16), current_thread);          } else if (reg == CPSR_REGISTER) { -            Core::CurrentArmInterface().SetCPSR(GdbHexToInt(buffer_ptr + i * 16)); +            RegWrite(CPSR_REGISTER, GdbHexToInt(buffer_ptr + i * 16), current_thread);          } else {              UNIMPLEMENTED();          } @@ -734,7 +843,7 @@ static void WriteMemory() {  void Break(bool is_memory_break) {      if (!halt_loop) {          halt_loop = true; -        SendSignal(SIGTRAP); +        send_trap = true;      }      memory_break = is_memory_break; @@ -744,10 +853,10 @@ void Break(bool is_memory_break) {  static void Step() {      step_loop = true;      halt_loop = true; -    step_break = true; -    SendSignal(SIGTRAP); +    send_trap = true;  } +/// Tell the CPU if we hit a memory breakpoint.  bool IsMemoryBreak() {      if (IsConnected()) {          return false; @@ -759,7 +868,6 @@ bool IsMemoryBreak() {  /// Tell the CPU to continue executing.  static void Continue() {      memory_break = false; -    step_break = false;      step_loop = false;      halt_loop = false;  } @@ -898,7 +1006,7 @@ void HandlePacket() {          HandleSetThread();          break;      case '?': -        SendSignal(latest_signal); +        SendSignal(current_thread, latest_signal);          break;      case 'k':          Shutdown(); @@ -935,6 +1043,9 @@ void HandlePacket() {      case 'Z':          AddBreakpoint();          break; +    case 'T': +        HandleThreadAlive(); +        break;      default:          SendReply("");          break; @@ -1079,4 +1190,11 @@ bool GetCpuStepFlag() {  void SetCpuStepFlag(bool is_step) {      step_loop = is_step;  } + +void SendTrap(Kernel::Thread* thread, int trap) { +    if (send_trap) { +        send_trap = false; +        SendSignal(thread, trap); +    } +}  }; // namespace GDBStub diff --git a/src/core/gdbstub/gdbstub.h b/src/core/gdbstub/gdbstub.h index 201fca095..f2418c9e4 100644 --- a/src/core/gdbstub/gdbstub.h +++ b/src/core/gdbstub/gdbstub.h @@ -7,6 +7,7 @@  #pragma once  #include "common/common_types.h" +#include "core/hle/kernel/thread.h"  namespace GDBStub { @@ -91,4 +92,12 @@ bool GetCpuStepFlag();   * @param is_step   */  void SetCpuStepFlag(bool is_step); + +/** + * Send trap signal from thread back to the gdbstub server. + * + * @param thread Sending thread. + * @param trap Trap no. + */ +void SendTrap(Kernel::Thread* thread, int trap);  } // namespace GDBStub diff --git a/src/video_core/engines/shader_bytecode.h b/src/video_core/engines/shader_bytecode.h index 8d4ea3401..4eb507325 100644 --- a/src/video_core/engines/shader_bytecode.h +++ b/src/video_core/engines/shader_bytecode.h @@ -173,6 +173,13 @@ enum class SubOp : u64 {      Min = 0x8,  }; +enum class FloatRoundingOp : u64 { +    None = 0, +    Floor = 1, +    Ceil = 2, +    Trunc = 3, +}; +  union Instruction {      Instruction& operator=(const Instruction& instr) {          value = instr.value; @@ -230,22 +237,19 @@ union Instruction {              std::memcpy(&result, &imm, sizeof(imm));              return result;          } -    } alu; -    union { -        BitField<39, 5, u64> shift_amount; -        BitField<20, 19, u64> immediate_low; -        BitField<56, 1, u64> immediate_high; -        BitField<48, 1, u64> negate_b; -        BitField<49, 1, u64> negate_a; - -        s32 GetImmediate() const { -            u32 immediate = static_cast<u32>(immediate_low | (immediate_high << 19)); +        s32 GetSignedImm20_20() const { +            u32 immediate = static_cast<u32>(imm20_19 | (negate_imm << 19));              // Sign extend the 20-bit value.              u32 mask = 1U << (20 - 1);              return static_cast<s32>((immediate ^ mask) - mask);          } +    } alu; +    union { +        BitField<39, 5, u64> shift_amount; +        BitField<48, 1, u64> negate_b; +        BitField<49, 1, u64> negate_a;      } iscadd;      union { @@ -293,11 +297,20 @@ union Instruction {      union {          BitField<10, 2, Register::Size> size; -        BitField<13, 1, u64> is_signed; +        BitField<12, 1, u64> is_output_signed; +        BitField<13, 1, u64> is_input_signed;          BitField<41, 2, u64> selector;          BitField<45, 1, u64> negate_a;          BitField<49, 1, u64> abs_a;          BitField<50, 1, u64> saturate_a; + +        union { +            BitField<39, 2, FloatRoundingOp> rounding; +        } f2i; + +        union { +            BitField<39, 4, u64> rounding; +        } f2f;      } conversion;      union { @@ -328,15 +341,16 @@ union Instruction {      } texs;      union { -        BitField<20, 5, u64> target; +        BitField<20, 24, u64> target;          BitField<5, 1, u64> constant_buffer;          s32 GetBranchTarget() const {              // Sign extend the branch target offset -            u32 mask = 1U << (5 - 1); +            u32 mask = 1U << (24 - 1);              u32 value = static_cast<u32>(target); -            // The branch offset is relative to the next instruction, so add 1 to it. -            return static_cast<s32>((value ^ mask) - mask) + 1; +            // The branch offset is relative to the next instruction and is stored in bytes, so +            // divide it by the size of an instruction and add 1 to it. +            return static_cast<s32>((value ^ mask) - mask) / sizeof(Instruction) + 1;          }      } bra; @@ -402,6 +416,9 @@ public:          MOV_R,          MOV_IMM,          MOV32_IMM, +        SHL_C, +        SHL_R, +        SHL_IMM,          SHR_C,          SHR_R,          SHR_IMM, @@ -424,6 +441,7 @@ public:          Trivial,          Arithmetic,          Logic, +        Shift,          ScaledAdd,          Ffma,          Flow, @@ -558,20 +576,23 @@ private:              INST("0100110010101---", Id::F2F_C, Type::Conversion, "F2F_C"),              INST("0101110010101---", Id::F2F_R, Type::Conversion, "F2F_R"),              INST("0011100-10101---", Id::F2F_IMM, Type::Conversion, "F2F_IMM"), -            INST("0100110010110---", Id::F2I_C, Type::Arithmetic, "F2I_C"), -            INST("0101110010110---", Id::F2I_R, Type::Arithmetic, "F2I_R"), -            INST("0011100-10110---", Id::F2I_IMM, Type::Arithmetic, "F2I_IMM"), +            INST("0100110010110---", Id::F2I_C, Type::Conversion, "F2I_C"), +            INST("0101110010110---", Id::F2I_R, Type::Conversion, "F2I_R"), +            INST("0011100-10110---", Id::F2I_IMM, Type::Conversion, "F2I_IMM"),              INST("0100110010011---", Id::MOV_C, Type::Arithmetic, "MOV_C"),              INST("0101110010011---", Id::MOV_R, Type::Arithmetic, "MOV_R"),              INST("0011100-10011---", Id::MOV_IMM, Type::Arithmetic, "MOV_IMM"),              INST("000000010000----", Id::MOV32_IMM, Type::Arithmetic, "MOV32_IMM"), -            INST("0100110000101---", Id::SHR_C, Type::Arithmetic, "SHR_C"), -            INST("0101110000101---", Id::SHR_R, Type::Arithmetic, "SHR_R"), -            INST("0011100-00101---", Id::SHR_IMM, Type::Arithmetic, "SHR_IMM"),              INST("0100110001100---", Id::FMNMX_C, Type::Arithmetic, "FMNMX_C"),              INST("0101110001100---", Id::FMNMX_R, Type::Arithmetic, "FMNMX_R"),              INST("0011100-01100---", Id::FMNMX_IMM, Type::Arithmetic, "FMNMX_IMM"),              INST("000001----------", Id::LOP32I, Type::Logic, "LOP32I"), +            INST("0100110001001---", Id::SHL_C, Type::Shift, "SHL_C"), +            INST("0101110001001---", Id::SHL_R, Type::Shift, "SHL_R"), +            INST("0011100-01001---", Id::SHL_IMM, Type::Shift, "SHL_IMM"), +            INST("0100110000101---", Id::SHR_C, Type::Shift, "SHR_C"), +            INST("0101110000101---", Id::SHR_R, Type::Shift, "SHR_R"), +            INST("0011100-00101---", Id::SHR_IMM, Type::Shift, "SHR_IMM"),              INST("0100110011100---", Id::I2I_C, Type::Conversion, "I2I_C"),              INST("0101110011100---", Id::I2I_R, Type::Conversion, "I2I_R"),              INST("01110001-1000---", Id::I2I_IMM, Type::Conversion, "I2I_IMM"), diff --git a/src/video_core/renderer_opengl/gl_shader_decompiler.cpp b/src/video_core/renderer_opengl/gl_shader_decompiler.cpp index eb8d37c9b..4a41e7798 100644 --- a/src/video_core/renderer_opengl/gl_shader_decompiler.cpp +++ b/src/video_core/renderer_opengl/gl_shader_decompiler.cpp @@ -893,6 +893,35 @@ private:              }              break;          } + +        case OpCode::Type::Shift: { +            std::string op_a = regs.GetRegisterAsInteger(instr.gpr8, 0, false); +            std::string op_b; + +            if (instr.is_b_imm) { +                op_b += '(' + std::to_string(instr.alu.GetSignedImm20_20()) + ')'; +            } else { +                if (instr.is_b_gpr) { +                    op_b += regs.GetRegisterAsInteger(instr.gpr20); +                } else { +                    op_b += regs.GetUniform(instr.uniform, GLSLRegister::Type::Integer); +                } +            } + +            switch (opcode->GetId()) { +            case OpCode::Id::SHL_C: +            case OpCode::Id::SHL_R: +            case OpCode::Id::SHL_IMM: +                regs.SetRegisterToInteger(instr.gpr0, true, 0, op_a + " << " + op_b, 1, 1); +                break; +            default: { +                NGLOG_CRITICAL(HW_GPU, "Unhandled shift instruction: {}", opcode->GetName()); +                UNREACHABLE(); +            } +            } +            break; +        } +          case OpCode::Type::ScaledAdd: {              std::string op_a = regs.GetRegisterAsInteger(instr.gpr8); @@ -902,12 +931,12 @@ private:              std::string op_b = instr.iscadd.negate_b ? "-" : "";              if (instr.is_b_imm) { -                op_b += '(' + std::to_string(instr.iscadd.GetImmediate()) + ')'; +                op_b += '(' + std::to_string(instr.alu.GetSignedImm20_20()) + ')';              } else {                  if (instr.is_b_gpr) {                      op_b += regs.GetRegisterAsInteger(instr.gpr20);                  } else { -                    op_b += regs.GetUniform(instr.uniform, instr.gpr0); +                    op_b += regs.GetUniform(instr.uniform, GLSLRegister::Type::Integer);                  }              } @@ -962,18 +991,20 @@ private:                  ASSERT_MSG(!instr.conversion.selector, "Unimplemented");                  std::string op_a = -                    regs.GetRegisterAsInteger(instr.gpr20, 0, instr.conversion.is_signed); +                    regs.GetRegisterAsInteger(instr.gpr20, 0, instr.conversion.is_input_signed);                  if (instr.conversion.abs_a) {                      op_a = "abs(" + op_a + ')';                  } -                regs.SetRegisterToInteger(instr.gpr0, instr.conversion.is_signed, 0, op_a, 1, 1); +                regs.SetRegisterToInteger(instr.gpr0, instr.conversion.is_output_signed, 0, op_a, 1, +                                          1);                  break;              }              case OpCode::Id::I2F_R: { +                ASSERT_MSG(!instr.conversion.selector, "Unimplemented");                  std::string op_a = -                    regs.GetRegisterAsInteger(instr.gpr20, 0, instr.conversion.is_signed); +                    regs.GetRegisterAsInteger(instr.gpr20, 0, instr.conversion.is_input_signed);                  if (instr.conversion.abs_a) {                      op_a = "abs(" + op_a + ')'; @@ -983,6 +1014,8 @@ private:                  break;              }              case OpCode::Id::F2F_R: { +                // TODO(Subv): Implement rounding operations. +                ASSERT_MSG(instr.conversion.f2f.rounding == 0, "Unimplemented rounding operation");                  std::string op_a = regs.GetRegisterAsFloat(instr.gpr20);                  if (instr.conversion.abs_a) { @@ -992,6 +1025,43 @@ private:                  regs.SetRegisterToFloat(instr.gpr0, 0, op_a, 1, 1);                  break;              } +            case OpCode::Id::F2I_R: { +                std::string op_a = regs.GetRegisterAsFloat(instr.gpr20); + +                if (instr.conversion.abs_a) { +                    op_a = "abs(" + op_a + ')'; +                } + +                using Tegra::Shader::FloatRoundingOp; +                switch (instr.conversion.f2i.rounding) { +                case FloatRoundingOp::None: +                    break; +                case FloatRoundingOp::Floor: +                    op_a = "floor(" + op_a + ')'; +                    break; +                case FloatRoundingOp::Ceil: +                    op_a = "ceil(" + op_a + ')'; +                    break; +                case FloatRoundingOp::Trunc: +                    op_a = "trunc(" + op_a + ')'; +                    break; +                default: +                    NGLOG_CRITICAL(HW_GPU, "Unimplemented f2i rounding mode {}", +                                   static_cast<u32>(instr.conversion.f2i.rounding.Value())); +                    UNREACHABLE(); +                    break; +                } + +                if (instr.conversion.is_output_signed) { +                    op_a = "int(" + op_a + ')'; +                } else { +                    op_a = "uint(" + op_a + ')'; +                } + +                regs.SetRegisterToInteger(instr.gpr0, instr.conversion.is_output_signed, 0, op_a, 1, +                                          1); +                break; +            }              default: {                  NGLOG_CRITICAL(HW_GPU, "Unhandled conversion instruction: {}", opcode->GetName());                  UNREACHABLE();  | 
