[libcxx-commits] [PATCH] D103400: [libcxx] [test] Fix the include_as_c.sh.cpp test in MSVC configurations

Martin Storsjö via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Mon May 31 12:13:56 PDT 2021


This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rG23f7f56cecf9: [libcxx] [test] Fix the include_as_c.sh.cpp test in MSVC configurations (authored by mstorsjo).

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D103400/new/

https://reviews.llvm.org/D103400

Files:
  libcxx/test/libcxx/include_as_c.sh.cpp


Index: libcxx/test/libcxx/include_as_c.sh.cpp
===================================================================
--- libcxx/test/libcxx/include_as_c.sh.cpp
+++ libcxx/test/libcxx/include_as_c.sh.cpp
@@ -14,8 +14,6 @@
 // file as C, but we're passing C++ flags on the command-line.
 // UNSUPPORTED: gcc
 
-// XFAIL: LIBCXX-WINDOWS-FIXME
-
 // Test that the C wrapper headers can be included when compiling them as C.
 
 // NOTE: It's not common or recommended to have libc++ in the header search
@@ -43,7 +41,11 @@
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
-#include <tgmath.h>
+// The clang-shipped tgmath.h header doesn't work with MSVC/UCRT's complex
+// headers in C mode, see PR46207.
+#ifndef _MSC_VER
+#    include <tgmath.h>
+#endif
 #include <wchar.h>
 #include <wctype.h>
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D103400.348849.patch
Type: text/x-patch
Size: 804 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20210531/1a6802a6/attachment.bin>


More information about the libcxx-commits mailing list