[PATCH] D56731: Add -Wimplicit-ctad warning to diagnose CTAD on types with no user defined deduction guides.
Geoffrey Romer via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jan 15 11:54:02 PST 2019
gromer added inline comments.
================
Comment at: include/clang/Basic/DiagnosticSemaKinds.td:2129
+def warn_class_template_argument_deduction_no_user_defined_guides : Warning<
+ "using class template argument deduction for %0 that has no user-defined deduction guides" >,
+ InGroup<ImplicitCTADUsage>, DefaultIgnore;
----------------
I'd prefer a phrasing more along the lines of "using class template argument deduction for %0 that might not intentionally support it". That gives us more room to do things like add an attribute later if necessary, and it helps the user understand why this warning indicates a potential problem.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D56731/new/
https://reviews.llvm.org/D56731
More information about the cfe-commits
mailing list