[PATCH] D18221: [Concepts] Implement subsection [dcl.spec.concept]p7 of the Concepts TS
Aaron Ballman via cfe-commits
cfe-commits at lists.llvm.org
Mon Mar 28 08:37:12 PDT 2016
aaron.ballman added a comment.
Generally looks good to me, but you should wait for @rsmith as he may have other ideas on the error recovery.
================
Comment at: lib/Sema/SemaDecl.cpp:6295
@@ +6294,3 @@
+ Diag(NewVD->getLocation(), diag::err_concept_specialized)
+ << 1 << (IsPartialSpecialization ? 2 : 1);
+ Diag(VarTmpl->getLocation(), diag::note_previous_declaration);
----------------
Can you put comments near the magic numbers to describe what they mean? e.g., `1 /*variable*/`
Same comment applies elsewhere in the patch as well.
http://reviews.llvm.org/D18221
More information about the cfe-commits
mailing list