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

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

2013-03-16 Thread Jason Merrill
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. Tested x86_64-pc-linux-gnu, applying to trunk. commit 02d2b645f935749d9f860cba2fab05656edbbce7 Author: Jason Merrill