Re: [PATCH v2] debug/93751 Option to generate DIEs for external variables - ping

2020-02-28 Thread Jason Merrill
On 2/28/20 4:12 PM, Alexey Neyman wrote: On 2/28/20 12:28 PM, Jason Merrill wrote: On 2/28/20 2:11 PM, Alexander Monakov wrote: On Fri, 28 Feb 2020, Eric Botcazou wrote: +1 - I think Alexey correctly pointed out in the Bugzilla that debuginfo growth from this change would be minimal

Re: [PATCH v2] debug/93751 Option to generate DIEs for external variables - ping

2020-02-28 Thread Alexey Neyman
On 2/28/20 12:28 PM, Jason Merrill wrote: On 2/28/20 2:11 PM, Alexander Monakov wrote: On Fri, 28 Feb 2020, Eric Botcazou wrote: +1 - I think Alexey correctly pointed out in the Bugzilla that debuginfo growth from this change would be minimal (usually the number of global variables is

Re: [PATCH v2] debug/93751 Option to generate DIEs for external variables - ping

2020-02-28 Thread Jason Merrill
On 2/28/20 2:11 PM, Alexander Monakov wrote: On Fri, 28 Feb 2020, Eric Botcazou wrote: +1 - I think Alexey correctly pointed out in the Bugzilla that debuginfo growth from this change would be minimal (usually the number of global variables is very small compared to number of functions (and

Re: [PATCH v2] debug/93751 Option to generate DIEs for external variables - ping

2020-02-28 Thread Alexander Monakov
On Fri, 28 Feb 2020, Eric Botcazou wrote: > > +1 - I think Alexey correctly pointed out in the Bugzilla that debuginfo > > growth from this change would be minimal (usually the number of global > > variables is very small compared to number of functions (and linemap info). > > Well, this will

Re: [PATCH v2] debug/93751 Option to generate DIEs for external variables - ping

2020-02-28 Thread Eric Botcazou
> +1 - I think Alexey correctly pointed out in the Bugzilla that debuginfo > growth from this change would be minimal (usually the number of global > variables is very small compared to number of functions (and linemap info). Well, this will drag the associated types too, so figures would be

Re: [PATCH v2] debug/93751 Option to generate DIEs for external variables - ping

2020-02-26 Thread Richard Biener
On February 26, 2020 8:26:06 PM GMT+01:00, Alexander Monakov wrote: >On Wed, 26 Feb 2020, Jason Merrill wrote: > >> Don't we want to fix the DWARF behavior to match the documentation? > >+1 - I think Alexey correctly pointed out in the Bugzilla that >debuginfo >growth from this change would be

Re: [PATCH v2] debug/93751 Option to generate DIEs for external variables - ping

2020-02-26 Thread Alexander Monakov
On Wed, 26 Feb 2020, Jason Merrill wrote: > Don't we want to fix the DWARF behavior to match the documentation? +1 - I think Alexey correctly pointed out in the Bugzilla that debuginfo growth from this change would be minimal (usually the number of global variables is very small compared to

Re: [PATCH v2] debug/93751 Option to generate DIEs for external variables - ping

2020-02-26 Thread Jason Merrill
On 2/26/20 2:02 PM, Alexey Neyman wrote: Patch ping. On 2/19/20 3:30 PM, Alexey Neyman wrote: Hi all, Attached is a patch adjusted per discussion in https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93751 - The manual is corrected to reflect that DIEs for external variables are not generated

Re: [PATCH v2] debug/93751 Option to generate DIEs for external variables - ping

2020-02-26 Thread Alexey Neyman
Patch ping. On 2/19/20 3:30 PM, Alexey Neyman wrote: Hi all, Attached is a patch adjusted per discussion in https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93751 - The manual is corrected to reflect that DIEs for external variables are not generated when using DWARF - A new option is

[PATCH v2] debug/93751 Option to generate DIEs for external variables

2020-02-19 Thread Alexey Neyman
Hi all, Attached is a patch adjusted per discussion in https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93751 - The manual is corrected to reflect that DIEs for external variables are not generated when using DWARF - A new option is introduced that implements the behavior that was described in