dmpots wrote: @JDevlieghere @jimingham @clayborg
Thanks for the comments. I updated the PR based on the initial feedback. We still have a few open issues that could use a bit more feedback: 1. Are we ok to start with a static table of plugin namespaces or do we want dynamic registration from the beginning? https://github.com/llvm/llvm-project/pull/134418#discussion_r2029164327 2. Are we ok adding the disabled plug-in info to the "statistics dump" output, or do we want it as part of the version dump? https://github.com/llvm/llvm-project/pull/134418#issuecomment-2784151621 3. Do we want a separate command for this, or should we do this with "settings set"? https://github.com/llvm/llvm-project/pull/134418#pullrequestreview-2747662941 Please take a look when you get a chance. Thanks! https://github.com/llvm/llvm-project/pull/134418 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
