[clang] c42116c - Fix up ms-pch-macro.c test to pass on non-Windows

Reid Kleckner via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 14 22:20:00 PST 2020


Author: Reid Kleckner
Date: 2020-01-14T22:19:30-08:00
New Revision: c42116cc653ae1618cc49dab367d9f6addd8cfd9

URL: https://github.com/llvm/llvm-project/commit/c42116cc653ae1618cc49dab367d9f6addd8cfd9
DIFF: https://github.com/llvm/llvm-project/commit/c42116cc653ae1618cc49dab367d9f6addd8cfd9.diff

LOG: Fix up ms-pch-macro.c test to pass on non-Windows

Added: 
    

Modified: 
    clang/test/PCH/ms-pch-macro.c

Removed: 
    


################################################################################
diff  --git a/clang/test/PCH/ms-pch-macro.c b/clang/test/PCH/ms-pch-macro.c
index eb45be1b6b73..3a8052eb322e 100644
--- a/clang/test/PCH/ms-pch-macro.c
+++ b/clang/test/PCH/ms-pch-macro.c
@@ -8,16 +8,16 @@
 // RUN: not %clang_cc1 -UFOO -DBAR=int -include-pch %t1.pch %s -pch-through-header=%S/variables.h 2> %t.err
 // RUN: FileCheck -check-prefix=CHECK-NOFOO %s < %t.err
 
-// RUN: %clang_cc1 -include-pch %t1.pch -DBAR=int -pch-through-header=%S/variables.h -verify %s 2> %t.err
+// RUN: %clang_cc1 -include-pch %t1.pch -DBAR=int -pch-through-header=%S/variables.h -verify %s
 
 // Enabling MS extensions should allow us to add BAR definitions.
-// RUN: %clang_cc1 -fms-extensions -DFOO %S/variables.h -emit-pch -o %t1.pch
-// RUN: %clang_cc1 -fms-extensions -include-pch %t1.pch -DBAR=int -pch-through-header=%S/variables.h -verify %s 2> %t.err
+// RUN: %clang_cc1 -DMSEXT -fms-extensions -DFOO %S/variables.h -emit-pch -o %t1.pch
+// RUN: %clang_cc1 -DMSEXT -fms-extensions -include-pch %t1.pch -DBAR=int -pch-through-header=%S/variables.h -verify %s
 
 #include "variables.h"
 
 BAR bar = 17;
-#ifndef _MSC_EXTENSIONS
+#ifndef MSEXT
 // expected-error at -2 {{unknown type name 'BAR'}}
 #endif
 
@@ -29,7 +29,7 @@ BAR bar = 17;
 #  error FOO has the wrong definition
 #endif
 
-#if defined(_MSC_EXTENSIONS) && !defined(BAR)
+#if defined(MSEXT) && !defined(BAR)
 #  error BAR was not defined
 #endif
 


        


More information about the cfe-commits mailing list