[PATCH] D13787: [clang-tidy] add check cppcoreguidelines-pro-type-vararg-use

Matthias Gehre via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 20 15:37:27 PDT 2015


mgehre added a comment.

Thanks for the good review!


================
Comment at: clang-tidy/cppcoreguidelines/ProTypeVarargCheck.cpp:26
@@ +25,3 @@
+  Finder->addMatcher(
+      varDecl(hasType(pointsTo(cxxRecordDecl(hasName("__va_list_tag")))))
+          .bind("va_list"),
----------------
sbenza wrote:
> Is there a way to look for the standard type (and not the implementation defined __xxx type name)?
I don't see it in the AST, because the other names are macros. But maybe there is some magic I don't know of.


http://reviews.llvm.org/D13787





More information about the cfe-commits mailing list