[clang] 28b26b1 - [clang] [test] Narrow down an MSVC specific behaviour to only not covever MinGW

Martin Storsjö via cfe-commits cfe-commits at lists.llvm.org
Tue May 30 13:06:49 PDT 2023


Author: Martin Storsjö
Date: 2023-05-30T23:05:57+03:00
New Revision: 28b26b161c2f5f8aecf8fffa7220cacc990ba51c

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

LOG: [clang] [test] Narrow down an MSVC specific behaviour to only not covever MinGW

This uses the same logic as in c2b256a990590dc8b69930259650cfeb085add03;
we can't check defined(_MSC_VER) invoked as %clang_cc1, therefore
check for !defined(__MINGW32__) instead.

This fixes the same issue in a new testcase that was added after this
issue was fixed last time in c2b256a990590dc8b69930259650cfeb085add03.

Differential Revision: https://reviews.llvm.org/D151661

Added: 
    

Modified: 
    clang/test/CXX/drs/dr9xx.cpp

Removed: 
    


################################################################################
diff  --git a/clang/test/CXX/drs/dr9xx.cpp b/clang/test/CXX/drs/dr9xx.cpp
index 4dfb98e5ec11e..fb13ef2967f88 100644
--- a/clang/test/CXX/drs/dr9xx.cpp
+++ b/clang/test/CXX/drs/dr9xx.cpp
@@ -92,7 +92,7 @@ namespace dr974 { // dr974: yes
 
 namespace dr977 { // dr977: yes
 enum E { e = E() };
-#ifndef _WIN32
+#if !defined(_WIN32) || defined(__MINGW32__)
 // expected-error at -2 {{invalid use of incomplete type 'E'}}
 // expected-note at -3 {{definition of 'dr977::E' is not complete until the closing '}'}}
 #endif


        


More information about the cfe-commits mailing list