Re: [PATCH] Remove misleading debug line entries

2020-12-10 Thread Richard Biener
On Wed, 9 Dec 2020, Bernd Edlinger wrote: > On 12/8/20 7:57 PM, Bernd Edlinger wrote: > > On 12/8/20 11:35 AM, Richard Biener wrote: > >> > >> + { > >> + /* Remove a nonbind marker when the outer scope of the > >> + inline function is completely removed. */ > >> +

Re: [PATCH] Remove misleading debug line entries

2020-12-09 Thread Bernd Edlinger
On 12/8/20 7:57 PM, Bernd Edlinger wrote: > On 12/8/20 11:35 AM, Richard Biener wrote: >> >> + { >> + /* Remove a nonbind marker when the outer scope of the >> + inline function is completely removed. */ >> + if (gimple_debug_nonbind_marker_p (stmt) >> +

Re: [PATCH] Remove misleading debug line entries

2020-12-08 Thread Bernd Edlinger
On 12/8/20 11:35 AM, Richard Biener wrote: > > + { > + /* Remove a nonbind marker when the outer scope of the > + inline function is completely removed. */ > + if (gimple_debug_nonbind_marker_p (stmt) > + && BLOCK_ABSTRACT_ORIGIN (b)) > +

Re: [PATCH] Remove misleading debug line entries

2020-12-08 Thread Richard Biener
On Mon, 7 Dec 2020, Bernd Edlinger wrote: > On 12/7/20 11:50 AM, Richard Biener wrote: > > The ipa-param-manipulation.c hunk is OK, please commit separately. > > > > done. > > > The tree-inline.c and cfgexpand.c changes are OK as well, for the > > tree-ssa-live.c change see below > > > > > >

Re: [PATCH] Remove misleading debug line entries

2020-12-07 Thread Bernd Edlinger
On 12/7/20 11:50 AM, Richard Biener wrote: > The ipa-param-manipulation.c hunk is OK, please commit separately. > done. > The tree-inline.c and cfgexpand.c changes are OK as well, for the > tree-ssa-live.c change see below > > > From 85b0e37d0c0d3ecac4908ebbfd67edc612ef22b2 Mon Sep 17 00:00:00

Re: [PATCH] Remove misleading debug line entries

2020-12-07 Thread Richard Biener
On Fri, 4 Dec 2020, Bernd Edlinger wrote: > On 12/3/20 9:30 AM, Richard Biener wrote: > > On Wed, 2 Dec 2020, Bernd Edlinger wrote: > > > >> On 12/2/20 8:50 AM, Richard Biener wrote: > >>> On Tue, 1 Dec 2020, Bernd Edlinger wrote: > >>> > Hi! > > > This removes gimple_debug st

Re: [PATCH] Remove misleading debug line entries

2020-12-04 Thread Bernd Edlinger
On 12/3/20 9:30 AM, Richard Biener wrote: > On Wed, 2 Dec 2020, Bernd Edlinger wrote: > >> On 12/2/20 8:50 AM, Richard Biener wrote: >>> On Tue, 1 Dec 2020, Bernd Edlinger wrote: >>> Hi! This removes gimple_debug stmts without block info after a NULL INLINE_ENTRY. >>>

Re: [PATCH] Remove misleading debug line entries

2020-12-03 Thread Richard Biener
On Wed, 2 Dec 2020, Bernd Edlinger wrote: > On 12/2/20 8:50 AM, Richard Biener wrote: > > On Tue, 1 Dec 2020, Bernd Edlinger wrote: > > > >> Hi! > >> > >> > >> This removes gimple_debug stmts without block info after a > >> NULL INLINE_ENTRY. > >> > >> The line numbers from these stmts are from t

Re: [PATCH] Remove misleading debug line entries

2020-12-02 Thread Bernd Edlinger
On 12/2/20 8:50 AM, Richard Biener wrote: > On Tue, 1 Dec 2020, Bernd Edlinger wrote: > >> Hi! >> >> >> This removes gimple_debug stmts without block info after a >> NULL INLINE_ENTRY. >> >> The line numbers from these stmts are from the inline function, >> but since the inline function is complet

Re: [PATCH] Remove misleading debug line entries

2020-12-01 Thread Richard Biener
On Tue, 1 Dec 2020, Bernd Edlinger wrote: > Hi! > > > This removes gimple_debug stmts without block info after a > NULL INLINE_ENTRY. > > The line numbers from these stmts are from the inline function, > but since the inline function is completely optimized away, > there will be no DW_TAG_inlin