diff mbox

Demangler patch committed: Fix constructor names with ABI tags

Message ID 55CF5E31.8060906@oracle.com
State New
Headers show

Commit Message

Paolo Carlini Aug. 15, 2015, 3:43 p.m. UTC 
... I think your patch fixes c++/63887. The libiberty testsuite passes 
with the below applied.

Paolo.

//////////////////
diff mbox

Patch

Index: testsuite/demangle-expected
===================================================================
--- testsuite/demangle-expected	(revision 226910)
+++ testsuite/demangle-expected	(working copy)
@@ -4390,8 +4390,13 @@  f(std::string[abi:foo], std::string[abi:foo])
 _Z18IndirectExternCallIPU7stdcallU7regparmILi3EEFviiEiEvT_T0_S3_
 void IndirectExternCall<void ( regparm<3> stdcall*)(int, int), int>(void ( regparm<3> stdcall*)(int, int), int, void ( regparm<3> stdcall*)(int, int))
 # 
-# ABI tags used to confuse the constructor name calculation.
+# ABI tags used to confuse the constructor and destructor name calculation.
 --format=gnu-v3 --no-params
 _ZNSt8ios_base7failureB5cxx11C1EPKcRKSt10error_code
 std::ios_base::failure[abi:cxx11]::failure(char const*, std::error_code const&)
 std::ios_base::failure[abi:cxx11]::failure
+# https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63887
+--format=gnu-v3 --no-params
+_ZNSt8time_getB5cxx11D1Ev
+std::time_get[abi:cxx11]::~time_get()
+std::time_get[abi:cxx11]::~time_get