[PATCH] D25437: [clang-tidy] Fix template agrument false positives in unused-using-decls.
    Haojian Wu via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Tue Oct 11 03:45:53 PDT 2016
    
    
  
hokein added inline comments.
================
Comment at: test/clang-tidy/misc-unused-using-decls.cpp:191
+// n::N is using in the explicit template instantiations.
+template void h(n::M<N>* t);
----------------
aaron.ballman wrote:
> Can you add a test using a non-type template argument, like an integer literal? I believe that still causes the diagnostic, though do we want it to?
Yeah, I agree that we need support non-type template arguments in the long term, but it seems out of scope of this patch (Supporting all non-type template arguments needs more stuff), I have added to FIXME.
https://reviews.llvm.org/D25437
    
    
More information about the cfe-commits
mailing list