SFN+LVU+IEPM v4 (was: Re: Statement Frontier Notes, Location Views, and Inlined Entry Point Markers)

2017-11-09 Thread Alexandre Oliva
On Sep 30, 2017, Alexandre Oliva wrote: > On Aug 31, 2017, Alexandre Oliva wrote: >> On Aug 23, 2017, Richard Biener wrote: >>> Just separating the boilerplate changes out from the "meat" of the change >>> into a separate patch for easier reviewing would be nice. >> I've broken up the patch in

Re: Statement Frontier Notes, Location Views, and Inlined Entry Point Markers

2017-09-30 Thread Alexandre Oliva
On Aug 31, 2017, Alexandre Oliva wrote: > On Aug 23, 2017, Richard Biener wrote: >> Just separating the boilerplate changes out from the "meat" of the change >> into a separate patch for easier reviewing would be nice. > I've broken up the patch into a patchset with 10 patches. I've already >

Re: Statement Frontier Notes, Location Views, and Inlined Entry Point Markers

2017-09-20 Thread Alexandre Oliva
On Sep 7, 2017, Joseph Myers wrote: > On Fri, 25 Aug 2017, Alexandre Oliva wrote: >> for gcc/ChangeLog >> >> * common.opt (Wa, Wl, Wp, g, gz=): Add >> RejectNegative. >> (gno-column-info): Remove. >> (gcolumn-info): Drop RejectNegative. >> (gno-): New prefix. [...] > OK. Thanks, I've finally

Re: Statement Frontier Notes, Location Views, and Inlined Entry Point Markers

2017-09-07 Thread Joseph Myers
On Fri, 25 Aug 2017, Alexandre Oliva wrote: > for gcc/ChangeLog > > * common.opt (Wa, Wl, Wp, g, gz=): Add > RejectNegative. > (gno-column-info): Remove. > (gcolumn-info): Drop RejectNegative. > (gno-): New prefix. > (gno-record-gcc-switches): Remove. >

Re: Statement Frontier Notes, Location Views, and Inlined Entry Point Markers

2017-08-31 Thread Alexandre Oliva
On Aug 23, 2017, Richard Biener wrote: > Just separating the boilerplate changes out from the "meat" of the change > into a separate patch for easier reviewing would be nice. I've broken up the patch into a patchset with 10 patches. I've already posted the one that makes -g options negatable th

Re: Statement Frontier Notes, Location Views, and Inlined Entry Point Markers

2017-08-28 Thread Richard Biener
On Fri, Aug 25, 2017 at 4:26 PM, Alexandre Oliva wrote: > On Aug 23, 2017, Richard Biener wrote: > if they are not a problem up until here why care now? > >>> IIRC we do have a limit for VTA notes too, but there's a C++ testcase >>> (g++.dg/tree-ssa/pr14703.C) that expands and inlines fibona

Re: Statement Frontier Notes, Location Views, and Inlined Entry Point Markers

2017-08-25 Thread Alexandre Oliva
On Aug 23, 2017, Richard Biener wrote: > Just curious if, for example, --with-build-config=bootstrap-lto > --enable-languages=c > --disable-multilib and make -j1 shows any difference in time and/or peak > memory > use (the interesting peak memory use is that of the WPA phase). $ ../configure --

Re: Statement Frontier Notes, Location Views, and Inlined Entry Point Markers

2017-08-25 Thread Alexandre Oliva
On Aug 23, 2017, Richard Biener wrote: > On Wed, Aug 23, 2017 at 12:29 AM, Alexandre Oliva wrote: >> All debug options that support negation seem to have adopted this idiom; >> without it, the negated options end up misparsed as -g with an argument, >> and then set_debug_level complains that "no

Re: Statement Frontier Notes, Location Views, and Inlined Entry Point Markers

2017-08-25 Thread Alexandre Oliva
On Aug 23, 2017, Richard Biener wrote: >>> if they are not a problem up until here why care now? >> IIRC we do have a limit for VTA notes too, but there's a C++ testcase >> (g++.dg/tree-ssa/pr14703.C) that expands and inlines fibonacci template >> functions so deep, more than doubling the number

Re: Statement Frontier Notes, Location Views, and Inlined Entry Point Markers

2017-08-23 Thread Richard Biener
On Wed, Aug 23, 2017 at 12:29 AM, Alexandre Oliva wrote: > On Aug 21, 2017, Richard Biener wrote: > >> +gno-statement-frontiers >> +Common Driver RejectNegative Var(debug_nonbind_markers_p, 0) Init(2) >> +Don't enforce progressive recommended breakpoint locations. >> + >> +gstatement-frontiers >>

Re: Statement Frontier Notes, Location Views, and Inlined Entry Point Markers

2017-08-23 Thread Richard Biener
On Tue, Aug 22, 2017 at 10:43 PM, Alexandre Oliva wrote: > On Aug 21, 2017, Richard Biener wrote: > >> On Fri, Aug 18, 2017 at 11:20 PM, Alexandre Oliva wrote: > >>> Besides implementing these new features, the patch contains multiple >>> fixes for -fcompare-debug errors detected at various opti

Re: Statement Frontier Notes, Location Views, and Inlined Entry Point Markers

2017-08-22 Thread Alexandre Oliva
On Aug 21, 2017, Richard Biener wrote: > +gno-statement-frontiers > +Common Driver RejectNegative Var(debug_nonbind_markers_p, 0) Init(2) > +Don't enforce progressive recommended breakpoint locations. > + > +gstatement-frontiers > +Common Driver RejectNegative Var(debug_nonbind_markers_p, 1) > +E

Re: Statement Frontier Notes, Location Views, and Inlined Entry Point Markers

2017-08-22 Thread Alexandre Oliva
On Aug 21, 2017, Richard Biener wrote: > On Fri, Aug 18, 2017 at 11:20 PM, Alexandre Oliva wrote: >> Besides implementing these new features, the patch contains multiple >> fixes for -fcompare-debug errors detected at various optimization >> levels, arising mainly from the introduction of begin