[PATCH] D11490: [Concepts] Add diagnostic: non template declaration

Nathan Wilson nwilson20 at gmail.com
Wed Jul 29 19:22:29 PDT 2015


Thanks Richard.

Sorry about the failed test that Hubert brought up. I'll remove that if no
one objects since it's covered by the Sema tests in this Patch.

Potentially that test in the Parser could be modified to be:
template<typename T>
concept D1 = true;

However, it will just pass and I believe it needs a diagnostic for the
return type not being bool.


On Wed, Jul 29, 2015 at 6:40 PM, Richard Smith <richard at metafoo.co.uk>
wrote:

> rsmith accepted this revision.
> rsmith added a comment.
> This revision is now accepted and ready to land.
>
> LGTM
>
>
> http://reviews.llvm.org/D11490
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150729/fe627b67/attachment.html>


More information about the cfe-commits mailing list