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 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
