Re: -Og bug? (was: [PATCH] libsanitizer demangling using cp-demangle.c)

2014-01-28 Thread Ian Lance Taylor
On Tue, Jan 28, 2014 at 6:36 AM, Thomas Schwinge wrote: > Avoid "'dc' may be uninitialized" warning. > > libiberty/ > * cp-demangle.c (d_demangle_callback): Put __builtin_unreachable > in place, to help the compiler. > > --- libiberty/cp-demangle.c > +++ libiberty/cp-demang

-Og bug? (was: [PATCH] libsanitizer demangling using cp-demangle.c)

2014-01-28 Thread Thomas Schwinge
Hi! This got committed to trunk as r206477; one small nit: On Tue, 10 Dec 2013 12:38:34 +0100, Jakub Jelinek wrote: > * libbacktrace/Makefile.am (libsanitizer_libbacktrace_la_SOURCES): Add > ../../libiberty/cp-demangle.c. Trying to build trunk r207180 with C*FLAGS='-Og -ggdb', a com

Re: [PATCH] libsanitizer demangling using cp-demangle.c

2014-01-16 Thread Alexey Samsonov
I've landed patches for libbacktrace and cp-demangle support in LLVM. However, they required some changes (e.g. some files LLVM trunk were modified after the last merge). This means that the next merge to GCC (IIUC it won't happen anytime soon before GCC 4.9 release) will not be clean. Sorry for de

Re: [PATCH] libsanitizer demangling using cp-demangle.c

2014-01-09 Thread Konstantin Serebryany
On Thu, Jan 9, 2014 at 5:57 PM, Jakub Jelinek wrote: > On Thu, Jan 09, 2014 at 05:51:05PM +0400, Konstantin Serebryany wrote: >> On Tue, Dec 10, 2013 at 3:38 PM, Jakub Jelinek wrote: >> > On Fri, Dec 06, 2013 at 06:40:52AM -0800, Ian Lance Taylor wrote: >> >> There was a recent buggy patch to the

Re: [PATCH] libsanitizer demangling using cp-demangle.c

2014-01-09 Thread Jakub Jelinek
On Thu, Jan 09, 2014 at 05:51:05PM +0400, Konstantin Serebryany wrote: > On Tue, Dec 10, 2013 at 3:38 PM, Jakub Jelinek wrote: > > On Fri, Dec 06, 2013 at 06:40:52AM -0800, Ian Lance Taylor wrote: > >> There was a recent buggy patch to the demangler that added calls to > >> malloc and realloc (201

Re: [PATCH] libsanitizer demangling using cp-demangle.c

2014-01-09 Thread Konstantin Serebryany
On Tue, Dec 10, 2013 at 3:38 PM, Jakub Jelinek wrote: > On Fri, Dec 06, 2013 at 06:40:52AM -0800, Ian Lance Taylor wrote: >> There was a recent buggy patch to the demangler that added calls to >> malloc and realloc (2013-10-25 Gary Benson ). >> That patch must be fixed or reverted before the 4.9 r

Re: [PATCH] libsanitizer demangling using cp-demangle.c

2014-01-09 Thread Dodji Seketeli
Jakub Jelinek a écrit: > > 2013-12-10 Jakub Jelinek > > * sanitizer_common/sanitizer_symbolizer_libbacktrace.h > (LibbacktraceSymbolizer::Demangle): New declaration. > * sanitizer_common/sanitizer_symbolizer_posix_libcdep.cc > (POSIXSymbolizer::Demangle): Use libbacktr

[PATCH] libsanitizer demangling using cp-demangle.c

2013-12-10 Thread Jakub Jelinek
On Fri, Dec 06, 2013 at 06:40:52AM -0800, Ian Lance Taylor wrote: > There was a recent buggy patch to the demangler that added calls to > malloc and realloc (2013-10-25 Gary Benson ). > That patch must be fixed or reverted before the 4.9 release. The main > code in the demangler must not call mall