Re: [PATCH] [PR82155] Fix crash in dwarf2out_abstract_function

2017-11-15 Thread Pierre-Marie de Rodat
On 11/15/2017 12:16 PM, Richard Biener wrote: Is it still okay to commit to gcc-7, now? Yes. Done. Thank you! -- Pierre-Marie de Rodat

Re: [PATCH] [PR82155] Fix crash in dwarf2out_abstract_function

2017-11-15 Thread Richard Biener
On Wed, Nov 15, 2017 at 10:11 AM, Pierre-Marie de Rodat wrote: > Hello Richard, > > On 09/25/2017 01:54 PM, Richard Biener wrote: >> >> Ok for trunk and gcc-7 branch after a while. > > Is it still okay to commit to gcc-7, now? Yes. Richard. > -- > Pierre-Marie de Rodat

Re: [PATCH] [PR82155] Fix crash in dwarf2out_abstract_function

2017-11-15 Thread Pierre-Marie de Rodat
Hello Richard, On 09/25/2017 01:54 PM, Richard Biener wrote: Ok for trunk and gcc-7 branch after a while. Is it still okay to commit to gcc-7, now? -- Pierre-Marie de Rodat

Re: [PATCH] [PR82155] Fix crash in dwarf2out_abstract_function

2017-09-25 Thread Pierre-Marie de Rodat
On 09/25/2017 01:54 PM, Richard Biener wrote: Ok for trunk and gcc-7 branch after a while. Thank you, Richard! Committed on trunk as 253147; I’ll wait at least one week to revisit the gcc-7 branch commit. -- Pierre-Marie de Rodat

Re: [PATCH] [PR82155] Fix crash in dwarf2out_abstract_function

2017-09-25 Thread Richard Biener
On Tue, Sep 12, 2017 at 8:00 AM, Pierre-Marie de Rodat wrote: > Hello, > > This patch is an attempt to fix the crash reported in PR82155. > > When generating a C++ class method for a class that is itself nested in > a class method, dwarf2out_early_global_decl currently leaves

[PATCH] [PR82155] Fix crash in dwarf2out_abstract_function

2017-09-12 Thread Pierre-Marie de Rodat
Hello, This patch is an attempt to fix the crash reported in PR82155. When generating a C++ class method for a class that is itself nested in a class method, dwarf2out_early_global_decl currently leaves the existing context DIE as it is if it already exists. However, it is possible that this