================
@@ -806,3 +807,21 @@ TEST_F(RISCVEmulatorTester, TestFMV_D_XInst) {
ASSERT_TRUE(this->Execute(*decode, false));
ASSERT_EQ(this->fpr.fpr[DecodeRD(FMV_D_XInst)], bits);
}
+
+TEST_F(RISCVEmulatorTester, TestGetRegisterInfoRV64) {
+ // Test that GetRegisterInfo returns valid register info for RV64
+ auto reg_info = this->GetRegisterInfo(eRegisterKindLLDB, gpr_x1_riscv);
+ ASSERT_TRUE(reg_info.has_value());
+ ASSERT_EQ(reg_info->byte_size, 8u);
+ ASSERT_STREQ(reg_info->name, "ra");
+}
+
+TEST_F(RISCVEmulatorTester32, TestGetRegisterInfoRV32) {
+ // Test that GetRegisterInfo returns valid register info for RV32
+ // This verifies the fix for issue #175092 where GetRegisterInfo was
+ // hardcoded to use RegisterInfoPOSIX_riscv64
----------------
JDevlieghere wrote:
Testing the desired behavior is sufficient. We can use git blame to figure out
why the test was added if anyone wants to find out.
```suggestion
// Test that GetRegisterInfo returns valid register info for riscv32.
```
https://github.com/llvm/llvm-project/pull/175262
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
