Re: [PR c+ 94827]: template parm with default requires

2020-04-30 Thread Nathan Sidwell
On 4/30/20 10:35 AM, Nathan Sidwell wrote: On 4/30/20 10:18 AM, Jason Merrill wrote: On 4/29/20 2:50 PM, Nathan Sidwell wrote: Jason, this is the patch you suggested, as I understood it.  I kept finish_nested_require's saving of the (converted) current_template_parms, becase of the comment ab

Re: [PR c+ 94827]: template parm with default requires

2020-04-30 Thread Nathan Sidwell
On 4/30/20 10:18 AM, Jason Merrill wrote: On 4/29/20 2:50 PM, Nathan Sidwell wrote: Jason, this is the patch you suggested, as I understood it.  I kept finish_nested_require's saving of the (converted) current_template_parms, becase of the comment about use in diagnostics. Is this what you m

Re: [PR c+ 94827]: template parm with default requires

2020-04-30 Thread Jason Merrill via Gcc-patches
On 4/29/20 2:50 PM, Nathan Sidwell wrote: Jason, this is the patch you suggested, as I understood it.  I kept finish_nested_require's saving of the (converted) current_template_parms, becase of the comment about use in diagnostics. Is this what you meant? Yes, this looks fine. But I don't

Re: [PR c+ 94827]: template parm with default requires

2020-04-30 Thread Nathan Sidwell
On 4/29/20 2:50 PM, Nathan Sidwell wrote: Jason, this is the patch you suggested, as I understood it.  I kept finish_nested_require's saving of the (converted) current_template_parms, becase of the comment about use in diagnostics. this extended test that tries to call the function ices wit

[PR c+ 94827]: template parm with default requires

2020-04-29 Thread Nathan Sidwell
Jason, this is the patch you suggested, as I understood it. I kept finish_nested_require's saving of the (converted) current_template_parms, becase of the comment about use in diagnostics. Is this what you meant? boostrapped on x86_64-linux. nathan -- Nathan Sidwell 2020-04-27 Jason Merril