[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