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