[libcxx] Amend XFAIL for failing tests on Windows

[petrhosek]

Some tests starting passing/failing after #69431 because Clang no longer enables -fdelayed-template-parsing by default on Windows with C++20.

[llvmbot]

@llvm/pr-subscribers-libcxx

Author: Petr Hosek (petrhosek)

Changes

Some tests starting passing/failing after #69431 because Clang no longer enables -fdelayed-template-parsing by default on Windows with C++20 modules.

---

Full diff: https://github.com/llvm/llvm-project/pull/70422.diff

4 Files Affected:

• (modified) libcxx/test/libcxx/atomics/diagnose_invalid_memory_order.verify.cpp (+1-1)
• (modified) libcxx/test/libcxx/fuzzing/random.pass.cpp (+5)
• (modified) libcxx/test/std/depr/depr.c.headers/math_h.pass.cpp (+5)
• (modified) libcxx/test/std/numerics/c.math/cmath.pass.cpp (+4)

diff --git a/libcxx/test/libcxx/atomics/diagnose_invalid_memory_order.verify.cpp b/libcxx/test/libcxx/atomics/diagnose_invalid_memory_order.verify.cpp
index a6c1160c7022f07..45c3f81fffd3d44 100644
--- a/libcxx/test/libcxx/atomics/diagnose_invalid_memory_order.verify.cpp
+++ b/libcxx/test/libcxx/atomics/diagnose_invalid_memory_order.verify.cpp
@@ -8,7 +8,7 @@
 
 // This test fails because diagnose_if doesn't emit all of the diagnostics
 // when -fdelayed-template-parsing is enabled, like it is in MSVC mode.
-// XFAIL: msvc
+// XFAIL: msvc && (clang-16 || clang-17)
 
 // REQUIRES: diagnose-if-support
 
diff --git a/libcxx/test/libcxx/fuzzing/random.pass.cpp b/libcxx/test/libcxx/fuzzing/random.pass.cpp
index 79ab7ac41151c97..21b7c9348c07821 100644
--- a/libcxx/test/libcxx/fuzzing/random.pass.cpp
+++ b/libcxx/test/libcxx/fuzzing/random.pass.cpp
@@ -6,6 +6,11 @@
 //
 //===----------------------------------------------------------------------===//
 
+// This test fails with because Clang no longer enables
+// -fdelayed-template-parsing by default on Windows with C++20 modules
+// (#69431).
+// XFAIL: msvc && clang-18
+
 // UNSUPPORTED: c++03, c++11
 
 #include <cassert>
diff --git a/libcxx/test/std/depr/depr.c.headers/math_h.pass.cpp b/libcxx/test/std/depr/depr.c.headers/math_h.pass.cpp
index 87767a2ee4311cb..eaa8b1ac44c7471 100644
--- a/libcxx/test/std/depr/depr.c.headers/math_h.pass.cpp
+++ b/libcxx/test/std/depr/depr.c.headers/math_h.pass.cpp
@@ -6,6 +6,11 @@
 //
 //===----------------------------------------------------------------------===//
 
+// This test fails with because Clang no longer enables
+// -fdelayed-template-parsing by default on Windows with C++20 modules (#69431).
+
+// XFAIL: msvc && clang-18
+
 // <math.h>
 
 #include <math.h>
diff --git a/libcxx/test/std/numerics/c.math/cmath.pass.cpp b/libcxx/test/std/numerics/c.math/cmath.pass.cpp
index 11a3de748cb7a69..ab7c759dcb438b2 100644
--- a/libcxx/test/std/numerics/c.math/cmath.pass.cpp
+++ b/libcxx/test/std/numerics/c.math/cmath.pass.cpp
@@ -6,6 +6,10 @@
 //
 //===----------------------------------------------------------------------===//
 
+// This test fails with because Clang no longer enables
+// -fdelayed-template-parsing by default on Windows with C++20 modules (#69431).
+// XFAIL: msvc && clang-18
+
 // <cmath>
 
 #include <cmath>

[ChuanqiXu9]

Clang no longer enables -fdelayed-template-parsing by default on Windows with C++20 modules.

Sorry for confusing. Actually that PR don't enable -fdelayed-template-parsing by default on Windows with C++20. It is not related to modules. I change the title in the commit messages: 366ffba too. But it looks like the github PR doesn't update automatically.

[petrhosek]

@ldionne @EricWF does this look good to you?

[petrhosek]

@ldionne @EricWF ping?

[ldionne]

Sorry for the delay, we had some dealings with an ABI break, then the week-end happened, etc. Thanks for fixing.