LLVM now has a copy of libcxxabi demangler, so lldb doesn't need to keep one too.
Diff Detail
Event Timeline
Cool. Would it be a good idea to only use LLVM's demangler? We can get rid of all the messy ifdef business, and I assume the LLVM demangler would be better maintained and more up to date than any system demangler.
We need to carefully benchmark any changes in demangling if we are thinking about switching to using llvm::itaniumDemangle() for everything. The demangler that was built into libcxxabi was 4 times slower than our fast version. I am not sure what version is now in LLVM, but I would guess that it might be the same version? FastDemangle is fast and any changes can't regress on performance.
AFAICT, this patch only changes the case where we were already not able to use the fast demangler. So nothing should change.
Yep, I saw that and that is why I OK'ed the patch. Someone else suggested cutting over to only use llvm::itaniumDemangle(), so thus my comment.
Glad to see this change. Maintaining another copy of the same demangler in LLDB is definitely unnecessary.