[lldb] [llvm] [lldb][RISCV] function calls support in lldb expressions (PR #99336)

via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 22 04:16:37 PDT 2024


dlav-sc wrote:

> > Thanks for the patches (if we end up going down this route you'll probably want to split out the LLVM JIT changes into a separate PR).
> 
> I know little about the JIT side but this is my first reaction too. I wonder if that has its own tests that need adding, and it'll get more expert eyes on it.
> 
> > without RISCV large code model, which has not been implemented yet
> 
> I presume the work to do this is much greater than the work shown here?

To be fair I'm not sure. We have this MR: https://github.com/riscv-non-isa/riscv-elf-psabi-doc/pull/388#issuecomment-1737115297 . This review is taking quite a while
already. So I presume there are some issue there. On the hand we can always disable the suggested path if needed.

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


More information about the llvm-commits mailing list