[Lldb-commits] [PATCH] D73018: [lldb] Add SystemInitializerAllPlugins and delete copy-pasted Init code in SystemInitializerFull and SystemInitializerTest
Pavel Labath via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Mon Jan 20 01:21:25 PST 2020
labath added a comment.
This seems like a good idea, but I really don't like the AllPlugins name, when it does not, in fact, initialize all plugins. I'm not really sure what to name it because criterion is pretty complex (though reasonable).
Another way to achieve something like this would be to create something like `Plugins/ObjectFile/AllPlugins.h` (or sth), which would initialize all object file plugins (and similar for other plugin kinds). Then the inividual "system initializers" would call these. That would still leave us with a bit of duplication between lldb-test and liblldb, but this could be justifiable as liblldb actually fully initializes some additional plugin kinds while lldb-test does not. (And we add new plugins much more often than we add a completely different plugin kinds).
Comment at: lldb/tools/lldb-test/CMakeLists.txt:20
I guess you don't need this then...
CHANGES SINCE LAST ACTION
More information about the lldb-commits