Re: [PATCH 4/9] [SFN] introduce statement frontier notes, still disabled

2017-11-01 Thread Jason Merrill
On Wed, Nov 1, 2017 at 3:13 PM, Alexandre Oliva wrote: > Hi, Jason, > > Apologies for the delay in responding; somehow I missed your reply when > it arrived (probably because I had a cold back then :-), and only saw it > today. > > On Oct 24, 2017, Jason Merrill

Re: [PATCH 4/9] [SFN] introduce statement frontier notes, still disabled

2017-11-01 Thread Alexandre Oliva
Hi, Jason, Apologies for the delay in responding; somehow I missed your reply when it arrived (probably because I had a cold back then :-), and only saw it today. On Oct 24, 2017, Jason Merrill wrote: > On Sat, Sep 30, 2017 at 5:08 AM, Alexandre Oliva

Re: [PATCH 4/9] [SFN] introduce statement frontier notes, still disabled

2017-10-24 Thread Jason Merrill
On Sat, Sep 30, 2017 at 5:08 AM, Alexandre Oliva wrote: > diff --git a/gcc/cp/constexpr.c b/gcc/cp/constexpr.c > index a89ee49..f9209ac 100644 > --- a/gcc/cp/constexpr.c > +++ b/gcc/cp/constexpr.c > @@ -306,6 +306,9 @@ build_data_member_initialization (tree t, >

Re: [PATCH 4/9] [SFN] introduce statement frontier notes, still disabled

2017-10-17 Thread Alexandre Oliva
On Oct 13, 2017, Richard Biener wrote: > If the [SFN] is self-contained you can install that part once the approval > for the FE parts is in. It is, so I'll do that. > You can of course wait a bit for more reviews > (stopped short on LVU because of that all-targets

Re: [PATCH 4/9] [SFN] introduce statement frontier notes, still disabled

2017-10-13 Thread Richard Biener
On Fri, Oct 13, 2017 at 9:21 AM, Alexandre Oliva wrote: > On Oct 9, 2017, Richard Biener wrote: > >> The middle-end changes are ok. The C FE change looks reasonable, >> I'd appreciate a 2nd look at the C++ FE changes by a maintainer. > >

Re: [PATCH 4/9] [SFN] introduce statement frontier notes, still disabled

2017-10-13 Thread Alexandre Oliva
On Oct 9, 2017, Richard Biener wrote: > The middle-end changes are ok. The C FE change looks reasonable, > I'd appreciate a 2nd look at the C++ FE changes by a maintainer. https://gcc.gnu.org/ml/gcc-patches/2017-09/msg02026.html Thanks a lot for the reviews!

Re: [PATCH 4/9] [SFN] introduce statement frontier notes, still disabled

2017-10-09 Thread Richard Biener
On Sat, Sep 30, 2017 at 11:08 AM, Alexandre Oliva wrote: > This patch completes the infrastructure for the introduction of > statement frontiers in C-family languages. > > It brings in all the code remaining code needed to introduce and > transform begin stmt trees, gimple

[PATCH 4/9] [SFN] introduce statement frontier notes, still disabled

2017-09-30 Thread Alexandre Oliva
This patch completes the infrastructure for the introduction of statement frontiers in C-family languages. It brings in all the code remaining code needed to introduce and transform begin stmt trees, gimple stmts, insns and notes, and ultimately use them to generate the is_stmt column in DWARF2+

[PATCH 4/9] [SFN] introduce statement frontier notes, still disabled

2017-08-31 Thread Alexandre Oliva
This patch completes the infrastructure for the introduction of statement frontiers in C-family languages. It brings in all the code remaining code needed to introduce and transform begin stmt trees, gimple stmts, insns and notes, and ultimately use them to generate the is_stmt column in DWARF2+