nickdesaulniers added inline comments.
================ Comment at: clang/test/Preprocessor/macro-reserved.c:22 +#undef __UINTPTR_TYPE__ +#undef __i386__ +#undef __UINT64_TYPE__ ---------------- This line wont have coverage unless the RUN line sets `-m32` (even then, an explicit triple would be better for non-x86 hosts running the test). Maybe add this `undef` to either init-x86.c or predefined-arch-macros.c? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D151741/new/ https://reviews.llvm.org/D151741 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits