[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