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

[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 it should default to off, for release
> branches I don't know).  The point is that while it is useful for
> people to report gcc bugs in production compilers, it is not useful for
> gcc developers on their boxes, having ICEs take 3 times as long is not
> desirable for people who deal with those all the time
We definitely want the ability to select.  Waiting on an ICE to trigger 
3 times would be painful for the developers.  So I'd think that enabled 
for the release branches and disabled for the trunk would be the right 
default.


> 2) recently a bug has been reported that some ICEs which print RTL
> to stderr are considered non-reproduceable, because the addresses e.g.
> in call_insn fndecl change through address space randomization.  For the
> retries the driver should supposedly append -fdump-noaddr.
Right.  I think we discussed this at Cauldron.

I think it's conceptually ok with those two changes, then it's just 
reviewing the details.

If you or Maxim could make those changes and repost, I'll cover the review.

jeff