Re: Ping: PATCH RFA: Print backtrace on ICE

2012-09-26 Thread Ian Lance Taylor
On Wed, Sep 26, 2012 at 1:52 PM, Dominique Dhumieres wrote: > > It also breaks the building of the Ada tools: > > ... > ../../xgcc -B../../ -I- -I../rts -I. -I/opt/gcc/work/gcc/ada -DIN_GCC -g > -O2 -W -Wall -o ../../gnatmake b_gnatm.o a-except.o ali.o ali-util.o > aspects.o s-casuti.o alloc.

Re: Ping: PATCH RFA: Print backtrace on ICE

2012-09-26 Thread Dominique Dhumieres
On Thu, Sep 20, 2012 at 10:33 PM, Ian Lance Taylor wrote: > Ping. > > The patch is the same, so I didn't attach it again. It may be found > at http://gcc.gnu.org/ml/gcc-patches/2012-09/msg01174.html . I think > I've replied to all comments. > It also breaks the building of the Ada tools: ... ..

Re: Ping: PATCH RFA: Print backtrace on ICE

2012-09-26 Thread Ian Lance Taylor
On Wed, Sep 26, 2012 at 8:32 AM, H.J. Lu wrote: > On Thu, Sep 20, 2012 at 10:33 PM, Ian Lance Taylor wrote: >> Ping. >> >> The patch is the same, so I didn't attach it again. It may be found >> at http://gcc.gnu.org/ml/gcc-patches/2012-09/msg01174.html . I think >> I've replied to all comments.

Re: Ping: PATCH RFA: Print backtrace on ICE

2012-09-26 Thread Rainer Orth
"H.J. Lu" writes: > On Thu, Sep 20, 2012 at 10:33 PM, Ian Lance Taylor wrote: >> Ping. >> >> The patch is the same, so I didn't attach it again. It may be found >> at http://gcc.gnu.org/ml/gcc-patches/2012-09/msg01174.html . I think >> I've replied to all comments. >> > > It breaks Linux/i686 b

Re: Ping: PATCH RFA: Print backtrace on ICE

2012-09-26 Thread H.J. Lu
On Thu, Sep 20, 2012 at 10:33 PM, Ian Lance Taylor wrote: > Ping. > > The patch is the same, so I didn't attach it again. It may be found > at http://gcc.gnu.org/ml/gcc-patches/2012-09/msg01174.html . I think > I've replied to all comments. > It breaks Linux/i686 build: ../../src-trunk/gcc/diag

Re: Ping: PATCH RFA: Print backtrace on ICE

2012-09-25 Thread Diego Novillo
On 2012-09-20 22:33 , Ian Lance Taylor wrote: gcc/: 2012-09-17 Ian Lance Taylor * diagnostic.c: Include "demangle.h" and "backtrace.h". (bt_stop): New static array. (bt_callback, bt_err_callback): New static functions. (diagnostic_action_after_output): Ca

Ping: PATCH RFA: Print backtrace on ICE

2012-09-20 Thread Ian Lance Taylor
ect: PATCH RFA: Print backtrace on ICE To: gcc-patches@gcc.gnu.org This patch to the diagnostic code uses the new backtrace library to print a backtrace on an ICE. For example, here is the output of a test case I took from a C++ PR: /home/iant/foo2.cc:6:6: internal compiler

Re: PATCH RFA: Print backtrace on ICE

2012-09-18 Thread Ian Lance Taylor
On Tue, Sep 18, 2012 at 7:55 AM, Richard Guenther wrote: > > Indeed - we ship binaries with stripped debug info, usually not installed. > libbacktrace will only produce useless garbage then. So I want a way > to disable it (at least by default) at configure time. The library won't print useless

Re: PATCH RFA: Print backtrace on ICE

2012-09-18 Thread Ian Lance Taylor
On Tue, Sep 18, 2012 at 7:40 AM, Markus Trippelsdorf wrote: > > If you use "make install-strip" to install, then libbacktrace will have > been build in vain. At least for this case a way to disable libbacktrace > should be available. Why bother? The library will fail to find anything useful to p

Re: PATCH RFA: Print backtrace on ICE

2012-09-18 Thread Markus Trippelsdorf
On 2012.09.18 at 16:55 +0200, Richard Guenther wrote: > On Tue, Sep 18, 2012 at 4:40 PM, Markus Trippelsdorf > wrote: > > On 2012.09.18 at 06:58 -0700, Ian Lance Taylor wrote: > >> On Tue, Sep 18, 2012 at 1:49 AM, Richard Guenther > >> wrote: > >> > On Mon, Sep 17, 2012 at 7:17 PM, Ian Lance Tayl

Re: PATCH RFA: Print backtrace on ICE

2012-09-18 Thread Richard Guenther
On Tue, Sep 18, 2012 at 4:40 PM, Markus Trippelsdorf wrote: > On 2012.09.18 at 06:58 -0700, Ian Lance Taylor wrote: >> On Tue, Sep 18, 2012 at 1:49 AM, Richard Guenther >> wrote: >> > On Mon, Sep 17, 2012 at 7:17 PM, Ian Lance Taylor wrote: >> >> OK for mainline? >> > >> > Hm. Can you please be

Re: PATCH RFA: Print backtrace on ICE

2012-09-18 Thread Markus Trippelsdorf
On 2012.09.18 at 06:58 -0700, Ian Lance Taylor wrote: > On Tue, Sep 18, 2012 at 1:49 AM, Richard Guenther > wrote: > > On Mon, Sep 17, 2012 at 7:17 PM, Ian Lance Taylor wrote: > >> OK for mainline? > > > > Hm. Can you please be that verbose only for ENABLE_CHECKING compilers? > > That would be

Re: PATCH RFA: Print backtrace on ICE

2012-09-18 Thread Ian Lance Taylor
On Tue, Sep 18, 2012 at 1:49 AM, Richard Guenther wrote: > On Mon, Sep 17, 2012 at 7:17 PM, Ian Lance Taylor wrote: >> This patch to the diagnostic code uses the new backtrace library to >> print a backtrace on an ICE. For example, here is the output of a test >> case I took from a C++ PR: >> >>

Re: PATCH RFA: Print backtrace on ICE

2012-09-18 Thread Richard Guenther
On Tue, Sep 18, 2012 at 10:49 AM, Richard Guenther wrote: > On Mon, Sep 17, 2012 at 7:17 PM, Ian Lance Taylor wrote: >> This patch to the diagnostic code uses the new backtrace library to >> print a backtrace on an ICE. For example, here is the output of a test >> case I took from a C++ PR: >> >

Re: PATCH RFA: Print backtrace on ICE

2012-09-18 Thread Richard Guenther
On Mon, Sep 17, 2012 at 7:17 PM, Ian Lance Taylor wrote: > This patch to the diagnostic code uses the new backtrace library to > print a backtrace on an ICE. For example, here is the output of a test > case I took from a C++ PR: > > /home/iant/foo2.cc:6:6: internal compiler error: in cp_lexer_new

PATCH RFA: Print backtrace on ICE

2012-09-17 Thread Ian Lance Taylor
This patch to the diagnostic code uses the new backtrace library to print a backtrace on an ICE. For example, here is the output of a test case I took from a C++ PR: /home/iant/foo2.cc:6:6: internal compiler error: in cp_lexer_new_from_tokens, at cp/parser.c:638 0xec549f internal_error(char cons