Re: [C++ Patch] PR 53305

2012-05-11 Thread Gabriel Dos Reis
On Fri, May 11, 2012 at 3:07 AM, Paolo Carlini wrote: > Thanks Gaby. Then I guess I'm going to commit the variant with NULL_TREE, I > like it a tad better because after all formally these functions return > trees. And consistently I change another instance only a few lines above. OK. Thanks; --

Re: [C++ Patch] PR 53305

2012-05-11 Thread Paolo Carlini
On 05/11/2012 04:48 AM, Gabriel Dos Reis wrote: On Thu, May 10, 2012 at 6:40 PM, Paolo Carlini wrote: Hi, an ICE on invalid (per Daniel's analysis): when r is NULL_TREE the next DECL_CONTEXT (r) can only crash. Plus a garbled error message because pp_cxx_simple_type_specifier doesn't handle BO

Re: [C++ Patch] PR 53305

2012-05-10 Thread Gabriel Dos Reis
On Thu, May 10, 2012 at 6:40 PM, Paolo Carlini wrote: > Hi, > > an ICE on invalid (per Daniel's analysis): when r is NULL_TREE the next > DECL_CONTEXT (r) can only crash. Plus a garbled error message because > pp_cxx_simple_type_specifier doesn't handle BOUND_TEMPLATE_TEMPLATE_PARM. > > Tested x86

[C++ Patch] PR 53305

2012-05-10 Thread Paolo Carlini
Hi, an ICE on invalid (per Daniel's analysis): when r is NULL_TREE the next DECL_CONTEXT (r) can only crash. Plus a garbled error message because pp_cxx_simple_type_specifier doesn't handle BOUND_TEMPLATE_TEMPLATE_PARM. Tested x86_64-linux. Thanks, Paolo. /// /cp 201