Re: [PATCH] Add testcase for c++/68116

2015-12-08 Thread Bernd Schmidt
On 12/08/2015 03:21 PM, Marek Polacek wrote: +C::T C::b[] +{ + T (&C::foo) +}; The problem I have with approving C++ testcases is that I have no idea whether this is valid or not or what it expresses. You should Cc Jason (which I've now done). That's odd code--I don't approve of the cast in

Re: [PATCH] Add testcase for c++/68116

2015-12-08 Thread Marek Polacek
On Tue, Dec 08, 2015 at 09:15:33AM -0500, Jason Merrill wrote: > On 12/08/2015 06:54 AM, Bernd Schmidt wrote: > >On 12/07/2015 06:49 PM, Marek Polacek wrote: > > > >>diff --git gcc/testsuite/g++.dg/cpp0x/pr68116.C > >>gcc/testsuite/g++.dg/cpp0x/pr68116.C > >>index e69de29..04ed901 100644 > >>--- gc

Re: [PATCH] Add testcase for c++/68116

2015-12-08 Thread Jason Merrill
On 12/08/2015 06:54 AM, Bernd Schmidt wrote: On 12/07/2015 06:49 PM, Marek Polacek wrote: diff --git gcc/testsuite/g++.dg/cpp0x/pr68116.C gcc/testsuite/g++.dg/cpp0x/pr68116.C index e69de29..04ed901 100644 --- gcc/testsuite/g++.dg/cpp0x/pr68116.C +++ gcc/testsuite/g++.dg/cpp0x/pr68116.C @@ -0,0

Re: [PATCH] Add testcase for c++/68116

2015-12-08 Thread Bernd Schmidt
On 12/07/2015 06:49 PM, Marek Polacek wrote: diff --git gcc/testsuite/g++.dg/cpp0x/pr68116.C gcc/testsuite/g++.dg/cpp0x/pr68116.C index e69de29..04ed901 100644 --- gcc/testsuite/g++.dg/cpp0x/pr68116.C +++ gcc/testsuite/g++.dg/cpp0x/pr68116.C @@ -0,0 +1,12 @@ +// PR c++/68116 +// { dg-do compile