aprantl added a comment. > I disagree that it is "carrying purity a little too far". My goal is to see > LLDB's non-plugin libraries be more language agnostic.
I have two motivations for this: > Better language support in LLDB, making it easier to add support for new > languages in a clean fashion. > Shrinking the size of lldb-server. Though the lldb on llvm.org doesn't > suffer from this problem too much, swift-lldb has an issue where the > lldb-server you build is incredibly bloated. On Linux I am measuring about > 162MB and on android I measured 61MB. It is incredibly difficult to shrink > the size without first decoupling components. Because swift-lldb is a > downstream project, I want to make sure that the abstractions there make > sense with what is on llvm.org, so I am doing this work here first. Those are good goals. Thank you for working on this! I don't yet see the connection between those goals and this patch, but I might be missing something. Would CPPLanguageRuntime need to be anything but a forward declaration in Process.h? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62755/new/ https://reviews.llvm.org/D62755 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
