[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