Re: C++ PATCH for c++/17232 (abstract class, array and sfinae)

2013-03-20 Thread Jason Merrill
And this is a robustness change that I ended up not needing, but still makes sense. commit 3099f644d5427e1db3b4041ca4ed779ef213b6a0 Author: Jason Merrill Date: Tue Mar 19 11:34:46 2013 -0400 * pt.c (retrieve_specialization): Handle null tmpl argument. diff --git a/gcc/cp/pt.c b/gcc/cp/pt

Re: C++ PATCH for c++/17232 (abstract class, array and sfinae)

2013-03-20 Thread Jason Merrill
On 03/17/2013 04:33 PM, Jason Merrill wrote: On 03/16/2013 03:38 PM, Jason Merrill wrote: In SFINAE context, we need to instantiate a class so that we can tell whether or not is abstract. Doing this in non-SFINAE context caused problems, so I've made it conditional. But it still causes proble

Re: C++ PATCH for c++/17232 (abstract class, array and sfinae)

2013-03-17 Thread Jason Merrill
On 03/16/2013 03:38 PM, Jason Merrill wrote: In SFINAE context, we need to instantiate a class so that we can tell whether or not is abstract. Doing this in non-SFINAE context caused problems, so I've made it conditional. But it still causes problems, such as PR 56642. So I've reverted the c