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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to