================ Comment at: lib/Headers/vadefs.h:24 @@ +23,3 @@ + +/* Only include this if we're compiling for the Windows platform. */ +#ifndef _MSC_VER ---------------- rnk wrote: > I'd wordsmith this to "Only include this if we're aiming for MSVC > compatibility." Done.
================ Comment at: lib/Headers/vadefs.h:32 @@ +31,3 @@ + +#include_next <vadefs.h> + ---------------- rnk wrote: > Should we only do this if __has_include(<vadefs.h>) ? I think we should always do it. If the user is including vadefs.h, she's expecting to get the system one, and we should just always try to forward to it. ================ Comment at: lib/Headers/vadefs.h:34 @@ +33,3 @@ + +/* Override macros from vadefs.h with definitions that work with Clang. */ +#define _crt_va_start(ap, param) __builtin_va_start(ap, param) ---------------- rnk wrote: > I think we should do "#ifndef _crt_va_start #undef _crt_va_start" to avoid > warnings with -Wsystem-headers. Done. http://reviews.llvm.org/D5784 _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
