[PATCH] D44826: Add -Wunused-using, a warning that finds unused using declarations.

Carlos Alberto Enciso via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 29 11:59:29 PDT 2018


CarlosAlbertoEnciso added inline comments.


================
Comment at: include/clang/Basic/DiagnosticGroups.td:828-829
                 // -Wunused-local-typedefs = -Wunused-local-typedef
+def : DiagGroup<"unused-usings", [UnusedUsing]>;
+                // -Wunused-usings = -Wunused-using
 
----------------
lebedev.ri wrote:
> Why? gcc compatibility?
No particular reason. I just follow the 'unused-local-typedefs' model.
If there is not objection from others reviewers, I will drop the gcc compatibility.


https://reviews.llvm.org/D44826





More information about the cfe-commits mailing list