[Lldb-commits] [PATCH] D10744: Fix demangling of names if required by language

Dawn Perchik dawn+llvm at burble.org
Tue Jul 7 11:59:04 PDT 2015


Greg, please respond.  Again, the solution you proposed will not work, because the language is unknown until the DWARF is read.  This patch has minimal impact on implementations for which this is not an issue, yet is critical, as it allows us to support Delphi and C++ in the same binary.   Please accept this patch after the issues you raised with in DWARFCompileUnit.cpp are addressed.

Is there anyway to appeal this?  Can other folks please have a look?  Someone with a better understanding of how the demangling is architected should be able to see why this patch is required.


Repository:
  rL LLVM

http://reviews.llvm.org/D10744







More information about the lldb-commits mailing list