efriedma added inline comments.
================ Comment at: lib/Headers/arm64intr.h:50 +#define _byteswap_ulong __builtin_bswap32 +#define _byteswap_uint64 __builtin_bswap64 + ---------------- These should be available for all Microsoft targets, I think. Please define these as inline functions or proper builtins, rather than macros; a macro is very likely to break something. (For example, if someone includes stdlib.h, which declares "unsigned long _byteswap_ulong(unsigned long);", replacing _byteswap_ulong with __builtin_bswap32 will cause a compile error.) Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56685/new/ https://reviews.llvm.org/D56685 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
