[PATCH] D51041: [clang-tidy] Don't run misc-unused-using-decls check in C++17.
    Gábor Horváth via Phabricator via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Tue Aug 21 08:19:44 PDT 2018
    
    
  
xazax.hun added inline comments.
================
Comment at: clang-tidy/misc/UnusedUsingDeclsCheck.cpp:35
+  // it is not ture in C++17's template argument deduction.
+  if (!getLangOpts().CPlusPlus || getLangOpts().CPlusPlus17 ||
+      getLangOpts().CPlusPlus2a)
----------------
Isn't the check for `getLangOpts().CPlusPlus2a` redundant? Shouldn't it imply the C++17 flag?
Repository:
  rCTE Clang Tools Extra
https://reviews.llvm.org/D51041
    
    
More information about the cfe-commits
mailing list