Re: C++ PATCH for c++/84978, ICE with NRVO

2018-03-20 Thread Jason Merrill
On Tue, Mar 20, 2018 at 11:55 AM, Jason Merrill wrote: > On Tue, Mar 20, 2018 at 9:32 AM, Marek Polacek wrote: >> We started crashing on this test with r258592 which added >> cp_get_callee_fndecl >> in in cp_genericize_r. >> >> This ICE apparently depends on whether we perform NRVO or not. If

Re: C++ PATCH for c++/84978, ICE with NRVO

2018-03-20 Thread Jason Merrill
On Tue, Mar 20, 2018 at 9:32 AM, Marek Polacek wrote: > We started crashing on this test with r258592 which added cp_get_callee_fndecl > in in cp_genericize_r. > > This ICE apparently depends on whether we perform NRVO or not. If the size of > S is <=16B we pass it in registers and it compiles f

C++ PATCH for c++/84978, ICE with NRVO

2018-03-20 Thread Marek Polacek
We started crashing on this test with r258592 which added cp_get_callee_fndecl in in cp_genericize_r. This ICE apparently depends on whether we perform NRVO or not. If the size of S is <=16B we pass it in registers and it compiles fine. But if the size of S is >16B, then we pass in memory, and