tzb99 added a comment.

In D128250#3661588 <https://reviews.llvm.org/D128250#3661588>, @Emmmer wrote:

> This commit updates:
>
> - Add the pc register according to the implementation of gdb 
> <https://github.com/riscv-collab/riscv-binutils-gdb/blob/riscv-binutils-2.38/gdb/features/riscv/64bit-cpu.c#L44>
> - Fix register type definition macros to pass register checks
>
> The NaN problem has been solved by D129750 <https://reviews.llvm.org/D129750>
>
> At this point, we can pass all LLDBUnitTest.
>
> It may be challenging to review and merge at one time for such a large patch. 
> I would like to ask if it‘s necessary to split this patch and merge them in 
> turn.

Hello:

Thank you so much for sharing the patch files. One thing I am still curious is 
what the ABISysv file you are using. Would you mind also sharing the remaining 
patches added for the lldb-server support?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D128250/new/

https://reviews.llvm.org/D128250

_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to