[Lldb-commits] [lldb] [LLDB][NFC] Move CPlusPlusLanguage methods used in Core/Module.cpp to a separated module to break lldb-server dependencies (PR #132274)

[Pavel Labath via lldb-commits]

labath wrote:

> Moving out only 1 function `CPlusPlusLanguage::IsCPPMangledName()` is not enough anymore to reduce the size of lldb-server. But this patch works fine because moves all code used by Module.cpp.

I still don't think we should be doing this, and this is the reason why. It's not a good solution if random perturbations in unrelated code can cause this to work or not.

> Note it is mainline clang/lld.

All the more reason to get to the bottom of this. Either we find a bug in the compiler/linker (and make it better for everyone), or we learn what is the property that makes this work (and then we can try to find a way to make it work reliably).

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