[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