nickdesaulniers accepted this revision. nickdesaulniers added a comment. This revision is now accepted and ready to land.
Thanks for the patch! ================ Comment at: clang/test/Preprocessor/undef-x86.c:2 +// RUN: %clang_cc1 -triple=i386-none-none -fsyntax-only -verify %s +// RUN: %clang_cc1 -triple=i686-none-none -fsyntax-only -verify %s +// RUN: %clang_cc1 -triple=x86_64-none-none -fsyntax-only -verify %s ---------------- I don't think i686 adds anything beyond i386, at least for this test they are both `-m32`; consider removing this run line. 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
