nickdesaulniers added inline comments.
================ Comment at: clang/test/Preprocessor/init-x86.c:1728-1741 +// RUN: %clang_cc1 -triple=i386-none-none -fsyntax-only -verify -DUNDEF %s +// RUN: %clang_cc1 -triple=i686-none-none -fsyntax-only -verify -DUNDEF %s +// RUN: %clang_cc1 -triple=x86_64-none-none -fsyntax-only -verify -DUNDEF %s +#ifdef UNDEF +// Check that we can undefine triple-specific defines without warning +// expected-no-diagnostics +#undef __i386 ---------------- Hmm...I guess if this is going to require adding run lines, then let's move these to a new test file in the same dir. Maybe `undef-x86.c` or whatever? Sorry for back and forth. 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
