[Lldb-commits] [lldb] Reland "[lldb] Reland 2402b3213c2f with `/H` to debug the windows build issue (PR #101672)

Pavel Labath via lldb-commits lldb-commits at lists.llvm.org
Fri Aug 16 00:34:10 PDT 2024


labath wrote:

Okay, that sort of makes sense. However, unless you actually want to be able to disable each of these plugins independently (which it sounds like you don't), then this is a very.. baroque way to achieve the desired effect (to have multiple (plugin) classes registered with the plugin manager).

All of that cmake plugin logic is just a very elaborate way to invoke `SomeClass::Initialize()`. Plugins are registered through `PluginManager::RegisterPlugin`. While the usual case is that the `Initialize` function performs a single `RegisterPlugin` call (and pretty much nothing else), nothing actually depends or enforces that, and we do have "plugins" where a single plugin library registers more than plugin class (e.g. `lldbPluginPlatformMacOSX`). I think it would be perfectly reasonable to register all of these classes from `ScriptInterpreterPythonInterfaces::Initialize` (and maybe even from `ScriptInterpreterPython::Initialize`).

https://github.com/llvm/llvm-project/pull/101672


More information about the lldb-commits mailing list