slydiman wrote:

> and that's why I am very reluctant to approve #132274

#132274 is most important now because it gives the significant effect. 
I have refactored `CPlusPlusLanguage::MethodName` and don't just splite a CU 
now. RichManglingContext.cpp uses auto 
`Language::FindPlugin(eLanguageTypeC_plus_plus)`, etc.

This patch is almost cosmetics. It removes unnecessary Debugger from 
lldb-server initialization chain. This speeds up and reduces memory usage fot 
now, but the effect is not noticeable.

So, I will move LoadPlugin lambda to SystemInitializerFull.

Note lldbInitialization depends on lldbPluginProcessGDBRemote, 
lldbPluginProcessPOSIX and lldbPluginProcessWindowsCommon. 
SystemInitializerCommon::Initialize() calls 
process_gdb_remote::ProcessGDBRemoteLog::Initialize(), 
ProcessPOSIXLog::Initialize() and ProcessWindowsLog::Initialize(). 
If it is OK for Utility, I will move lldbInitialization into lldbUtility.

https://github.com/llvm/llvm-project/pull/134383
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to