Re: C++ PATCH for c++/80095, ICE with this pointer in NSDMI

2017-04-07 Thread Jason Merrill
OK, thanks. On Wed, Apr 5, 2017 at 5:49 AM, Marek Polacek wrote: > Ping. > > On Wed, Mar 29, 2017 at 11:32:39PM +0200, Marek Polacek wrote: >> On Wed, Mar 29, 2017 at 02:56:51PM -0400, Jason Merrill wrote: >> > On Wed, Mar 29, 2017 at 12:38 PM, Marek Polacek wrote: >> > > Here we have a referenc

Re: C++ PATCH for c++/80095, ICE with this pointer in NSDMI

2017-04-05 Thread Marek Polacek
Ping. On Wed, Mar 29, 2017 at 11:32:39PM +0200, Marek Polacek wrote: > On Wed, Mar 29, 2017 at 02:56:51PM -0400, Jason Merrill wrote: > > On Wed, Mar 29, 2017 at 12:38 PM, Marek Polacek wrote: > > > Here we have a reference initialization with NSDMI and *this. We are > > > crashing > > > becaus

Re: C++ PATCH for c++/80095, ICE with this pointer in NSDMI

2017-03-29 Thread Marek Polacek
On Wed, Mar 29, 2017 at 02:56:51PM -0400, Jason Merrill wrote: > On Wed, Mar 29, 2017 at 12:38 PM, Marek Polacek wrote: > > Here we have a reference initialization with NSDMI and *this. We are > > crashing > > because a PLACEHOLDER_EXPR crept into the gimplifier. > > > > This happens since r2186

Re: C++ PATCH for c++/80095, ICE with this pointer in NSDMI

2017-03-29 Thread Jason Merrill
On Wed, Mar 29, 2017 at 12:38 PM, Marek Polacek wrote: > Here we have a reference initialization with NSDMI and *this. We are crashing > because a PLACEHOLDER_EXPR crept into the gimplifier. > > This happens since r218653 where set_up_extended_ref_temp was changed to > use split_nonconstant_init.

C++ PATCH for c++/80095, ICE with this pointer in NSDMI

2017-03-29 Thread Marek Polacek
Here we have a reference initialization with NSDMI and *this. We are crashing because a PLACEHOLDER_EXPR crept into the gimplifier. This happens since r218653 where set_up_extended_ref_temp was changed to use split_nonconstant_init. As a consequence, cp_gimplify_init_expr might now be receiving