Re: C++ PATCH to fix ICE in replace_placeholders_r (PR c++/79937)

2017-05-03 Thread Jason Merrill
On Tue, Apr 25, 2017 at 12:17 PM, Marek Polacek wrote: > On Fri, Apr 07, 2017 at 03:27:36PM -0400, Jason Merrill wrote: >> On Fri, Mar 24, 2017 at 12:22 PM, Marek Polacek wrote: >> > On Thu, Mar 23, 2017 at 05:09:58PM -0400, Jason Merrill wrote: >> >> On Thu, Mar 23, 2017 at 4:34 PM, Marek Polace

Re: C++ PATCH to fix ICE in replace_placeholders_r (PR c++/79937)

2017-04-25 Thread Marek Polacek
On Fri, Apr 07, 2017 at 03:27:36PM -0400, Jason Merrill wrote: > On Fri, Mar 24, 2017 at 12:22 PM, Marek Polacek wrote: > > On Thu, Mar 23, 2017 at 05:09:58PM -0400, Jason Merrill wrote: > >> On Thu, Mar 23, 2017 at 4:34 PM, Marek Polacek wrote: > >> > On Tue, Mar 14, 2017 at 02:34:30PM -0400, Ja

Re: C++ PATCH to fix ICE in replace_placeholders_r (PR c++/79937)

2017-04-07 Thread Jason Merrill
On Fri, Mar 24, 2017 at 12:22 PM, Marek Polacek wrote: > On Thu, Mar 23, 2017 at 05:09:58PM -0400, Jason Merrill wrote: >> On Thu, Mar 23, 2017 at 4:34 PM, Marek Polacek wrote: >> > On Tue, Mar 14, 2017 at 02:34:30PM -0400, Jason Merrill wrote: >> >> On Tue, Mar 14, 2017 at 2:33 PM, Jason Merrill

Re: C++ PATCH to fix ICE in replace_placeholders_r (PR c++/79937)

2017-04-03 Thread Marek Polacek
Ping. Any ideas how to move this forward? On Fri, Mar 24, 2017 at 05:22:00PM +0100, Marek Polacek wrote: > On Thu, Mar 23, 2017 at 05:09:58PM -0400, Jason Merrill wrote: > > On Thu, Mar 23, 2017 at 4:34 PM, Marek Polacek wrote: > > > On Tue, Mar 14, 2017 at 02:34:30PM -0400, Jason Merrill wrote:

Re: C++ PATCH to fix ICE in replace_placeholders_r (PR c++/79937)

2017-03-24 Thread Marek Polacek
On Thu, Mar 23, 2017 at 05:09:58PM -0400, Jason Merrill wrote: > On Thu, Mar 23, 2017 at 4:34 PM, Marek Polacek wrote: > > On Tue, Mar 14, 2017 at 02:34:30PM -0400, Jason Merrill wrote: > >> On Tue, Mar 14, 2017 at 2:33 PM, Jason Merrill wrote: > >> > On Tue, Mar 7, 2017 at 12:10 PM, Marek Polace

Re: C++ PATCH to fix ICE in replace_placeholders_r (PR c++/79937)

2017-03-23 Thread Jason Merrill
On Thu, Mar 23, 2017 at 4:34 PM, Marek Polacek wrote: > On Tue, Mar 14, 2017 at 02:34:30PM -0400, Jason Merrill wrote: >> On Tue, Mar 14, 2017 at 2:33 PM, Jason Merrill wrote: >> > On Tue, Mar 7, 2017 at 12:10 PM, Marek Polacek wrote: >> >> In this testcase we have >> >> C c = bar (X{1}); >> >>

Re: C++ PATCH to fix ICE in replace_placeholders_r (PR c++/79937)

2017-03-23 Thread Marek Polacek
On Tue, Mar 14, 2017 at 02:34:30PM -0400, Jason Merrill wrote: > On Tue, Mar 14, 2017 at 2:33 PM, Jason Merrill wrote: > > On Tue, Mar 7, 2017 at 12:10 PM, Marek Polacek wrote: > >> In this testcase we have > >> C c = bar (X{1}); > >> which store_init_value sees as > >> c = TARGET_EXPR >> .n=(&)

Re: C++ PATCH to fix ICE in replace_placeholders_r (PR c++/79937)

2017-03-14 Thread Jason Merrill
On Tue, Mar 14, 2017 at 2:33 PM, Jason Merrill wrote: > On Tue, Mar 7, 2017 at 12:10 PM, Marek Polacek wrote: >> In this testcase we have >> C c = bar (X{1}); >> which store_init_value sees as >> c = TARGET_EXPR > .n=(&)->i}>)> >> i.e. we're initializing "c" with a TARGET_EXPR. We call replace_p

Re: C++ PATCH to fix ICE in replace_placeholders_r (PR c++/79937)

2017-03-14 Thread Jason Merrill
On Tue, Mar 7, 2017 at 12:10 PM, Marek Polacek wrote: > In this testcase we have > C c = bar (X{1}); > which store_init_value sees as > c = TARGET_EXPR .n=(&)->i}>)> > i.e. we're initializing "c" with a TARGET_EXPR. We call replace_placeholders > that walks the whole tree to substitute the place

Re: C++ PATCH to fix ICE in replace_placeholders_r (PR c++/79937)

2017-03-14 Thread Marek Polacek
Ping. On Tue, Mar 07, 2017 at 06:10:48PM +0100, Marek Polacek wrote: > In this testcase we have > C c = bar (X{1}); > which store_init_value sees as > c = TARGET_EXPR .n=(&)->i}>)> > i.e. we're initializing "c" with a TARGET_EXPR. We call replace_placeholders > that walks the whole tree to subst

C++ PATCH to fix ICE in replace_placeholders_r (PR c++/79937)

2017-03-07 Thread Marek Polacek
In this testcase we have C c = bar (X{1}); which store_init_value sees as c = TARGET_EXPR )->i}>)> i.e. we're initializing "c" with a TARGET_EXPR. We call replace_placeholders that walks the whole tree to substitute the placeholders. Eventually we find the nested but that's for another object, s