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

[Alexey Merzlyakov via lldb-commits]

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

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

Another structure question is that: starting from this patch, LLDB for RISCV32 is switching to work over dynamic registers set, which is totally right and great. In this case, we do not need to have outdated `RegisterInfoPOSIX_riscv32` with static presets, while it will be switched to the new dynamic concept. So, why do not just rename (and/or merge) `RegisterInfoPOSIXDynamic_riscv32` with `RegisterInfoPOSIX_riscv32` leaving only its dynamic logic inside (`DynamicRegisterInfo m_dyn_reg_infos` and all related stuff)? In this case, we might need to make small adoptions also EmulateInstruction and may-be NativeRegisterContext to `SetRegisterInfo` with GPR+FPR registers presets so it would not change their logic. This way the entities will not duplicate.

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