JDevlieghere added a comment. In D62732#2724078 <https://reviews.llvm.org/D62732#2724078>, @luismarques wrote:
> In D62732#2724040 <https://reviews.llvm.org/D62732#2724040>, @JDevlieghere > wrote: > >> Any update on the testing strategy yet? It seems like that this is actively >> being used and worked on, so I'm generally supportive of landing this with >> some elementary coverage. > > A while ago I got a skeleton of lldb-server compiling for RISC-V, but it > seemed like it would be quite a bit of work to flesh out that implementation. > I can only allocate some amount of effort to that right now, so that part > might take a while to get finished. Thanks for the update! > I was going to follow up on @labath's suggestions of further testing just > this patch (the ObjectFileELF and the disassembler changes), but some other > work got in the way. That bit should be easier to prioritize, especially if > we agreed that such kind of testing would be enough to get this landed. Sounds reasonable to me. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62732/new/ https://reviews.llvm.org/D62732 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
