hans accepted this revision.
hans added a comment.
This revision is now accepted and ready to land.

Highly useful I'd say!

lgtm



================
Comment at: include/clang/Basic/Diagnostic.td:113
+// Extensions are warnings about accepted language extensions that are
+// default-off but enabled by -pedantic.
 class Extension<string str> : Diagnostic<str, CLASS_EXTENSION, SEV_Ignored>;
----------------
Nit: On first glance, one might parse this as the accepted language extensions 
being enabled by -pedantic, not the warnings. Maybe spell it out: "about 
accepted language extensions. The warnings are default-off bug enabled by 
-pedantic."


https://reviews.llvm.org/D32371



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

Reply via email to