chelcassanova wrote: One more time, I've updated the patch. Some points:
- I've removed our exception list for the methods that were unconditionally given a connection - For the FIXME that we had for changing the LLDB namespace I've changed to a note as to why we use simple string replacement. - Again, for the pointer plus len list, that can be addressed in a follow up PR. - I added a CMake variable to optionally skip building LLDB RPC. >From this, I think this patch is pretty much ready to land if no one else >finds big outstanding issues :) https://github.com/llvm/llvm-project/pull/138031 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits