[Lldb-commits] [lldb] [lldb][LoongArch] Complete register alias name in `AugmentRegisterInfo` (PR #124059)

via lldb-commits lldb-commits at lists.llvm.org
Thu Jan 23 22:17:21 PST 2025


wangleiat wrote:

> ABI details are someone else's job but on generic things this is the right idea.
> 
> It needs tests, I suggest you find the tests that read general registers and add aliased name reads to those. Though it's unlikely to break for just one use case, I would add this to core file and live process tests, since it's simple and cheap to do.

Sorry, I couldn't find any reference test cases. Since the Chinese Spring Festival holiday started today, the follow-up changes for this patch will have to wait until after the holiday(two weeks later). Or interested individuals can continue to improve it.

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


More information about the lldb-commits mailing list