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:
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
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
==