Re: [C++ PATCH] Fix ICE during VEC_INIT_EXPR gimplification (PR c++/77739)

2016-11-23 Thread Jason Merrill
On Wed, Nov 23, 2016 at 9:31 AM, Jakub Jelinek wrote: > On Wed, Nov 23, 2016 at 09:11:02AM -0500, Jason Merrill wrote: >> On Tue, Nov 22, 2016 at 3:45 PM, Jakub Jelinek wrote: >> > As mentioned in the PR, another option I see is special case >> > REFERENCE_REF_P INDIRECT_REFs and MEM_REFs into wh

Re: [C++ PATCH] Fix ICE during VEC_INIT_EXPR gimplification (PR c++/77739)

2016-11-23 Thread Jakub Jelinek
On Wed, Nov 23, 2016 at 09:11:02AM -0500, Jason Merrill wrote: > On Tue, Nov 22, 2016 at 3:45 PM, Jakub Jelinek wrote: > > As mentioned in the PR, another option I see is special case > > REFERENCE_REF_P INDIRECT_REFs and MEM_REFs into which they are gimplified > > in cp_genericize_r by not changi

Re: [C++ PATCH] Fix ICE during VEC_INIT_EXPR gimplification (PR c++/77739)

2016-11-23 Thread Jason Merrill
On Tue, Nov 22, 2016 at 3:45 PM, Jakub Jelinek wrote: > As mentioned in the PR, another option I see is special case > REFERENCE_REF_P INDIRECT_REFs and MEM_REFs into which they are gimplified > in cp_genericize_r by not changing is_invisiref_parm decls if they are > already wrapped in those. Tha

[C++ PATCH] Fix ICE during VEC_INIT_EXPR gimplification (PR c++/77739)

2016-11-22 Thread Jakub Jelinek
Hi! As mentioned in the PR, we ICE because part of the body is genericized twice and each time it wraps is_invisiref_parm RESULT_DECL (in this case, could be also PARM_DEC) into REFERENCE_REF_P INDIRECT_REF. The first time it is desirable, but when done again during VEC_INIT_EXPR gimplification wh