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

[PATCH] Add testcase for c++/68116

2015-12-07 Thread Marek Polacek
This testcase used to ICE, but compiles fine since the C++ delayed folding merge. I'd like to add it to the testsuite and close the PR. Tested on x86_64-linux, ok for trunk? 2015-12-07 Marek Polacek PR c++/68116 * g++.dg/cpp0x/pr68116.C: New test. diff --git gcc/testsuite/g+