nwilson added inline comments.

================
Comment at: include/clang/Basic/DiagnosticSemaKinds.td:2003
@@ +2002,3 @@
+def err_concept_specified_specialization : Error<
+  "%'concept' cannot be applied on an "
+  "%select{explicit instantiation|explicit specialization|partial 
specialization}0">;
----------------
aaron.ballman wrote:
> Is this an extraneous %?
Good catch. I'll remove it when making the commit.

================
Comment at: lib/Sema/SemaDecl.cpp:7659
@@ +7658,3 @@
+        Diag(D.getDeclSpec().getConceptSpecLoc(),
+             diag::err_concept_specified_specialization) << 1;
+      }
----------------
hubert.reinterpretcast wrote:
> I don't think the declaration should still be marked as a concept in this 
> case.
Hmm, Richard - did you have any thoughts about this? IIRC, we might be okay 
here by only looking at the concept flag of the primary template.


http://reviews.llvm.org/D13357



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to