Merge pull request #76 from Decompollaborate/develop

1.12.6

CHANGELOG.md

@@ -7,6 +7,13 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [1.12.6] - 2025-02-06
+
+### Fixed
+
+- R5900: Fix decoding the `imm5` operand of the `viaddi` instruction as a signed
+  value instead of decoding it as an unsigned one.
+
 ## [1.12.5] - 2024-12-16
 
 ### Fixed
@@ -702,6 +709,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - First version
 
 [unreleased]: https://github.com/Decompollaborate/rabbitizer/compare/master...develop
+[1.12.6]: https://github.com/Decompollaborate/rabbitizer/compare/1.12.5...1.12.6
 [1.12.5]: https://github.com/Decompollaborate/rabbitizer/compare/1.12.4...1.12.5
 [1.12.4]: https://github.com/Decompollaborate/rabbitizer/compare/1.12.3...1.12.4
 [1.12.3]: https://github.com/Decompollaborate/rabbitizer/compare/1.12.2...1.12.3

Cargo.lock

@@ -102,7 +102,7 @@ dependencies = [
 
 [[package]]
 name = "rabbitizer"
-version = "1.12.5"
+version = "1.12.6"
 dependencies = [
  "cc",
  "glob",

Cargo.toml

@@ -4,7 +4,7 @@
 [package]
 name = "rabbitizer"
 # Version should be synced with include/common/RabbitizerVersion.h
-version = "1.12.5"
+version = "1.12.6"
 edition = "2021"
 authors = ["Anghelo Carvajal <angheloalf95@gmail.com>"]
 description = "MIPS instruction decoder"

README.md

@@ -109,7 +109,7 @@ cargo add rabbitizer
 Or you can add it manually to your `Cargo.toml`:
 
 ```toml
-rabbitizer = "1.12.5"
+rabbitizer = "1.12.6"
 ```
 
 See this crate at <https://crates.io/crates/rabbitizer>.

cplusplus/include/instructions/InstructionR5900.hpp

@@ -33,7 +33,7 @@ namespace rabbitizer {
         uint8_t GetR5900_l() const;
         uint8_t GetR5900_m() const;
 
-        uint8_t GetR5900_imm5() const;
+        int8_t GetR5900_imm5() const;
     };
 };
 

cplusplus/src/instructions/InstructionR5900.cpp

@@ -113,6 +113,6 @@ uint8_t InstructionR5900::GetR5900_m() const {
     return RAB_INSTR_R5900_GET_m(&this->instr);
 }
 
-uint8_t InstructionR5900::GetR5900_imm5() const {
-    return RAB_INSTR_R5900_GET_imm5(&this->instr);
+int8_t InstructionR5900::GetR5900_imm5() const {
+    return RabbitizerUtils_From2Complement(RAB_INSTR_R5900_GET_imm5(&this->instr), 5);
 }

include/common/RabbitizerVersion.h

@@ -14,7 +14,7 @@ extern "C" {
 // Header version
 #define RAB_VERSION_MAJOR 1
 #define RAB_VERSION_MINOR 12
-#define RAB_VERSION_PATCH 5
+#define RAB_VERSION_PATCH 6
 
 #define RAB_VERSION_STR RAB_STRINGIFY(RAB_VERSION_MAJOR) "." RAB_STRINGIFY(RAB_VERSION_MINOR) "." RAB_STRINGIFY(RAB_VERSION_PATCH)
 

pyproject.toml

@@ -4,7 +4,7 @@
 [project]
 name = "rabbitizer"
 # Version should be synced with include/common/RabbitizerVersion.h
-version = "1.12.5"
+version = "1.12.6"
 description = "MIPS instruction decoder"
 # license = "MIT"
 readme = "README.md"

src/instructions/RabbitizerInstructionR5900/RabbitizerInstructionR5900_OperandType.c

@@ -515,7 +515,7 @@ size_t RabbitizerOperandType_process_r5900_immediate5(const RabbitizerInstructio
         return immOverrideLength;
     }
 
-    number = RAB_INSTR_R5900_GET_imm5(self);
+    number = RabbitizerUtils_From2Complement(RAB_INSTR_R5900_GET_imm5(self), 5);
     if (RabbitizerConfig_Cfg.misc.omit0XOnSmallImm) {
         if (number > -10 && number < 10) {
             RABUTILS_BUFFER_SPRINTF(dst, totalSize, "%i", number);

tests/c/instruction_checks/r5900_disasm.c

@@ -40,6 +40,8 @@ const TestEntry test_entries[] = {
     TEST_ENTRY_C(0x4A820BFF, NULL, "viswr.y     $vi2, ($vi1)"),
     TEST_ENTRY_C(0x4A420BFF, NULL, "viswr.z     $vi2, ($vi1)"),
     TEST_ENTRY_C(0x4A220BFF, NULL, "viswr.w     $vi2, ($vi1)"),
+
+    TEST_ENTRY_C(0x4A0307B2, NULL, "viaddi      $vi3, $vi0, -0x2"),
 };
 
 size_t test_entries_len = ARRAY_COUNT(test_entries);