Re: [PATCH 6/7] [D] libiberty: Improve support for demangling D2 templates

2015-05-16 Thread Iain Buclaw
On 14 May 2015 at 17:30, Iain Buclaw wrote: > On 14 May 2015 at 15:24, Jeff Law wrote: >> On 05/13/2015 02:51 AM, Iain Buclaw wrote: >>> >>> In my tests, this gives the demangler near-complete support. Of a >>> sample of about 75k symbols pulled from the standard library >>> unittester, all but

Re: [PATCH 6/7] [D] libiberty: Improve support for demangling D2 templates

2015-05-16 Thread Iain Buclaw
On 14 May 2015 at 19:47, Joseph Myers wrote: > On Thu, 14 May 2015, Iain Buclaw wrote: > >> On another note, I've found out why the remaining 20 symbols in my 75k >> sample failed. They don't fail at all! It's just that they were all >> greater than 33,000 characters in length, and my test used

Re: [PATCH 6/7] [D] libiberty: Improve support for demangling D2 templates

2015-05-14 Thread Mike Stump
On May 14, 2015, at 8:30 AM, Iain Buclaw wrote: > > On another note, I've found out why the remaining 20 symbols in my 75k > sample failed. They don't fail at all! It's just that they were all > greater than 33,000 characters in length, and my test used c++filt, > which trims anything bigger th

Re: [PATCH 6/7] [D] libiberty: Improve support for demangling D2 templates

2015-05-14 Thread Joseph Myers
On Thu, 14 May 2015, Iain Buclaw wrote: > On another note, I've found out why the remaining 20 symbols in my 75k > sample failed. They don't fail at all! It's just that they were all > greater than 33,000 characters in length, and my test used c++filt, > which trims anything bigger than 32767 (w

Re: [PATCH 6/7] [D] libiberty: Improve support for demangling D2 templates

2015-05-14 Thread Iain Buclaw
On 14 May 2015 at 15:24, Jeff Law wrote: > On 05/13/2015 02:51 AM, Iain Buclaw wrote: >> >> In my tests, this gives the demangler near-complete support. Of a >> sample of about 75k symbols pulled from the standard library >> unittester, all but 20 were successfully parsed. >> >> --- >> libiberty/

Re: [PATCH 6/7] [D] libiberty: Improve support for demangling D2 templates

2015-05-14 Thread Iain Buclaw
On 14 May 2015 at 15:24, Jeff Law wrote: > On 05/13/2015 02:51 AM, Iain Buclaw wrote: >> >> In my tests, this gives the demangler near-complete support. Of a >> sample of about 75k symbols pulled from the standard library >> unittester, all but 20 were successfully parsed. >> >> --- >> libiberty/

Re: [PATCH 6/7] [D] libiberty: Improve support for demangling D2 templates

2015-05-14 Thread Jeff Law
On 05/13/2015 02:51 AM, Iain Buclaw wrote: In my tests, this gives the demangler near-complete support. Of a sample of about 75k symbols pulled from the standard library unittester, all but 20 were successfully parsed. --- libiberty/ChangeLog: 2015-05-13 Iain Buclaw * d-demangle.c (dlang

[PATCH 6/7] [D] libiberty: Improve support for demangling D2 templates

2015-05-13 Thread Iain Buclaw
In my tests, this gives the demangler near-complete support. Of a sample of about 75k symbols pulled from the standard library unittester, all but 20 were successfully parsed. --- libiberty/ChangeLog: 2015-05-13 Iain Buclaw * d-demangle.c (dlang_symbol_kinds): New enum. (dlang_parse_s