[PATCH] D42085: Add va_start()/va_copy()/va_end to Builtins.def

2018-01-16 Thread Nico Weber via Phabricator via cfe-commits
thakis closed this revision. thakis added a comment. r322573, thanks! Yes, I looked at the same file and came to the same conclusion regarding a test :-) https://reviews.llvm.org/D42085 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http:

[PATCH] D42085: Add va_start()/va_copy()/va_end to Builtins.def

2018-01-16 Thread Hans Wennborg via Phabricator via cfe-commits
hans accepted this revision. hans added a comment. This revision is now accepted and ready to land. lgtm Seems that some of these are tested in Sema/implicit-builtin-decl.c, but it doesn't look exhaustive and I'm not sure it's worth adding these. https://reviews.llvm.org/D42085

[PATCH] D42085: Add va_start()/va_copy()/va_end to Builtins.def

2018-01-15 Thread Nico Weber via Phabricator via cfe-commits
thakis created this revision. thakis added a reviewer: hans. That way, clang suggests including stdarg.h when these are used in C files. https://reviews.llvm.org/D42085 Files: include/clang/Basic/Builtins.def Index: include/clang/Basic/Builtins.def ==