Re: Fwd: [Bug debug/53754] [4.8 Regression][lto] ICE in lhd_decl_printable_name, at langhooks.c:222 (with -g)

2012-06-29 Thread Cary Coutant
>> That's sensible anyways - you avoid useless work.  So the patch is ok. > > OK, I'll submit this patch, but leave the PR open and follow up with > something to fix the underlying problem as you suggest. Here's the patch I'm committing (as soon as bootstrap and testing completes)... -cary 2012

Re: Fwd: [Bug debug/53754] [4.8 Regression][lto] ICE in lhd_decl_printable_name, at langhooks.c:222 (with -g)

2012-06-29 Thread Cary Coutant
> > (What's the right way to send a patch to fix a PR? I'm not even sure > > whether you were cc'ed on my response.) > > The right way to send a patch to fix a PR is to send it to gcc-patches ;) Well, yeah, but in this case I was just proposing a patch for discussion, and using the bugzilla mechan

Re: Fwd: [Bug debug/53754] [4.8 Regression][lto] ICE in lhd_decl_printable_name, at langhooks.c:222 (with -g)

2012-06-29 Thread Richard Guenther
On Thu, 28 Jun 2012, Cary Coutant wrote: > [resending in plain text. Sorry, gmail defaulted to HTML.] > > Ping. I'm not looking for commit approval yet, just advice on how > thorough we need to be to support -g and LTO together. > > (What's the right way to send a patch to fix a PR? I'm not even

Fwd: [Bug debug/53754] [4.8 Regression][lto] ICE in lhd_decl_printable_name, at langhooks.c:222 (with -g)

2012-06-28 Thread Cary Coutant
[resending in plain text. Sorry, gmail defaulted to HTML.] Ping. I'm not looking for commit approval yet, just advice on how thorough we need to be to support -g and LTO together. (What's the right way to send a patch to fix a PR? I'm not even sure whether you were cc'ed on my response.) -cary