Re: [PATCH][Ping v5] Add patch for debugging compiler ICEs

2014-08-04 Thread Maxim Ostapenko
Thanks Jeff and Jakub, I've reposted ICE debugging patch into gcc-patches mailing list (https://gcc.gnu.org/ml/gcc-patches/2014-08/msg00285.html). -Maxim On 08/01/2014 11:29 PM, Jeff Law wrote: On 08/01/14 02:00, Jakub Jelinek wrote: On Thu, Jul 24, 2014 at 04:39:28PM +0400, Maxim Ostapenko w

Re: [PATCH][Ping v5] Add patch for debugging compiler ICEs

2014-08-03 Thread Yury Gribov
On 08/01/2014 07:53 PM, Jakub Jelinek wrote: I think we should use David Malcolm's approach i.e. add some --report-bug flag to driver. This could be enabled by default at configure time via --with-spec. -freport-bug or whatever we call it should not be, at least if it attempts to communicate ove

Re: [PATCH][Ping v5] Add patch for debugging compiler ICEs

2014-08-01 Thread Jeff Law
On 08/01/14 02:00, Jakub Jelinek wrote: On Thu, Jul 24, 2014 at 04:39:28PM +0400, Maxim Ostapenko wrote: Ping. Don't want to review a patch I wrote partially, so just a few comments: 1) IMHO it should be configure time selectable (not sure about the default, but for non-release branches IMHO i

Re: [PATCH][Ping v5] Add patch for debugging compiler ICEs

2014-08-01 Thread Jakub Jelinek
On Fri, Aug 01, 2014 at 12:13:18PM +0400, Yury Gribov wrote: > On 08/01/2014 12:00 PM, Jakub Jelinek wrote: > >Don't want to review a patch I wrote partially, so just a few comments: > >1) IMHO it should be configure time selectable (not sure about the default, > >but for non-release branches IMHO

Re: [PATCH][Ping v5] Add patch for debugging compiler ICEs

2014-08-01 Thread Jakub Jelinek
On Fri, Aug 01, 2014 at 08:43:27AM -0700, Andi Kleen wrote: > Jakub Jelinek writes: > > Don't want to review a patch I wrote partially, so just a few comments: > > 1) IMHO it should be configure time selectable (not sure about the default, > > but for non-release branches IMHO it should default to

Re: [PATCH][Ping v5] Add patch for debugging compiler ICEs

2014-08-01 Thread Andi Kleen
Jakub Jelinek writes: > Don't want to review a patch I wrote partially, so just a few comments: > 1) IMHO it should be configure time selectable (not sure about the default, > but for non-release branches IMHO it should default to off, for release > branches I don't know). The point is that while

Re: [PATCH][Ping v5] Add patch for debugging compiler ICEs

2014-08-01 Thread Yury Gribov
On 08/01/2014 12:00 PM, Jakub Jelinek wrote: Don't want to review a patch I wrote partially, so just a few comments: 1) IMHO it should be configure time selectable (not sure about the default, but for non-release branches IMHO it should default to off, for release branches I don't know). The poi

Re: [PATCH][Ping v5] Add patch for debugging compiler ICEs

2014-08-01 Thread Jakub Jelinek
On Thu, Jul 24, 2014 at 04:39:28PM +0400, Maxim Ostapenko wrote: > Ping. Don't want to review a patch I wrote partially, so just a few comments: 1) IMHO it should be configure time selectable (not sure about the default, but for non-release branches IMHO it should default to off, for release branc

[PATCH][Ping v5] Add patch for debugging compiler ICEs

2014-07-24 Thread Maxim Ostapenko
Ping. Original Message Subject:[PATCH][Ping v4] Add patch for debugging compiler ICEs Date: Fri, 11 Jul 2014 17:44:28 +0400 From: Maxim Ostapenko To: GCC Patches CC: Yury Gribov , Slava Garbuzov , Jakub Jelinek , tsaund...@mozilla.com, Maxim Ostapenko P