[Lldb-commits] [lldb] [lldb][RISCV] Handle subsets of CSRs in RV32 core dump images (PR #142932)

[Alexey Merzlyakov via lldb-commits]

================

----------------
AlexeyMerzlyakov wrote:

Typically the `RegisterContextPOSIX_riscv32.h` contains general information about register contexts that might use in other places (like real-live debugging, not only Coredumps). So, I believe that the common methods like `InvalidateAllRegisters(), GetRegisterCount(), GetRegisterInfoAtIndex(), GetRegisterSetCount(), GetRegisterSet(), IsGPR(), IsFPR() and IsCSR()` better to be there, while coredump-related remaining logic to remain in `RegisterContextPOSIXCore_riscv32.h`. Mixing them will break existing concept of LLDB structure for other targets and may cause confusion in future development. So, I would like to re-factor the code to keep the above methods to `Context*` class scope.

https://github.com/llvm/llvm-project/pull/142932