[cfe-commits] [patch] fix for PR11216

Douglas Gregor dgregor at apple.com
Tue Oct 25 07:22:59 PDT 2011


On Oct 24, 2011, at 10:59 PM, David Blaikie wrote:

> Here's a fairly non-intrusive fix for PR11216: "Can't derive from a
> decltype-specifier" including a couple of test cases & some updated
> naming/comments.
> 
> Let me know if this looks OK & I'll check it in.

It looks pretty good, but I have two requests:

1) The "else" that handles class-names is really long. I think the flow would be better if decltype() was handled completely in its own "if" early on, since it shares so little code with the class-name case.

2) Please add some tests that have a dependent decltype expression, to make sure template instantiation works. I'm sure it will, but...

	- Doug




More information about the cfe-commits mailing list