[libcxx] [test] Change LIBCXX-WINDOWS-FIXME into XFAIL: msvc for cases that succeed in mingw configurations

[mstorsjo]

Summary

Add a FIXME comment for those instances that don't yet have an
accompanying explanation comment.

Also narrow down some "XFAIL: windows-dll" into "XFAIL: windows-dll &&
msvc", for things that don't fail correspondingly in mingw configurations.

[ldionne]

libcxx/test/std/language.support/support.dynamic/alloc.errors/set.new.handler/get_new_handler.pass.cpp

11–12

I don't understand why we're not keeping LIBCXX-WINDOWS-FIXME in those cases.

[mstorsjo]

libcxx/test/std/language.support/support.dynamic/alloc.errors/set.new.handler/get_new_handler.pass.cpp

11–12

Because LIBCXX-WINDOWS-FIXME is set in both msvc and mingw configurations, but the test succeeds in mingw configs, so the bug, whatever it is, is specific to msvc. When adding mingw CI, these tests give “unexpectedly passed” errors.

[mstorsjo]

Updated after splitting out other cases. Added/clarified comments about the reason for the XFAILs in the remaining cases.

[mstorsjo]

Rebase and rerun CI

[ldionne]

Thanks for adding comments, that was interesting to read. If someone wants to investigate finishing up Windows/MSVC support, they'll have an easier time.

libcxx/test/std/language.support/support.dynamic/alloc.errors/set.new.handler/get_new_handler.pass.cpp

 //===----------------------------------------------------------------------===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.
 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
 //
 //===----------------------------------------------------------------------===//
 
 // test get_new_handler
 
-// XFAIL: LIBCXX-WINDOWS-FIXME
+// FIXME: This XFAIL has no explanation yet.
+// XFAIL: msvc
 
 #include <new>
 #include <cassert>
 
 #include "test_macros.h"
 
 void f1() {}
 void f2() {}

libcxx/test/std/language.support/support.exception/except.nested/rethrow_if_nested.pass.cpp

 //===----------------------------------------------------------------------===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.
 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
 //
 //===----------------------------------------------------------------------===//
 
 // UNSUPPORTED: no-exceptions
 
 // This test fails due to a stack overflow
-// XFAIL: LIBCXX-WINDOWS-FIXME
+// XFAIL: msvc
 
 // <exception>
 
 // class nested_exception;
 
 // template <class E> void rethrow_if_nested(const E& e);
 
 #include <exception>

libcxx/test/std/language.support/support.exception/propagation/current_exception.pass.cpp

 //===----------------------------------------------------------------------===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.
 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
 //
 //===----------------------------------------------------------------------===//
 
 // This test needs to be rewritten for the Windows exception_ptr semantics
 // which copy the exception each time the exception_ptr is copied.
-// XFAIL: LIBCXX-WINDOWS-FIXME
+// XFAIL: msvc
 
 // UNSUPPORTED: no-exceptions
 // <exception>
 
 // exception_ptr current_exception();
 
 #include <exception>
 #include <cassert>

libcxx/test/std/localization/locale.categories/category.collate/locale.collate.byname/hash.pass.cpp

 //===----------------------------------------------------------------------===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.
 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
 //
 //===----------------------------------------------------------------------===//
 
 // REQUIRES: locale.en_US.UTF-8
 
 // https://llvm.org/PR41018
-// XFAIL: windows-dll
+// XFAIL: windows-dll && msvc
 
 // <locale>
 
 // template <class charT> class collate_byname
 
 // long hash(const charT* low, const charT* high) const;
 
 //   This test is not portable

libcxx/test/std/localization/locale.categories/category.collate/locale.collate/locale.collate.members/compare.pass.cpp

 //===----------------------------------------------------------------------===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.
 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
 //
 //===----------------------------------------------------------------------===//
 
 // https://llvm.org/PR41018
-// XFAIL: windows-dll
+// XFAIL: windows-dll && msvc
 
 // <locale>
 
 // template <class charT> class collate;
 
 // int compare(const charT* low1, const charT* high1,
 //             const charT* low2, const charT* high2) const;
 

libcxx/test/std/localization/locale.categories/category.collate/locale.collate/locale.collate.members/hash.pass.cpp

 //===----------------------------------------------------------------------===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.
 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
 //
 //===----------------------------------------------------------------------===//
 
 // https://llvm.org/PR41018
-// XFAIL: windows-dll
+// XFAIL: windows-dll && msvc
 
 // <locale>
 
 // template <class charT> class collate;
 
 // long hash(const charT* low, const charT* high) const;
 
 //   This test is not portable

libcxx/test/std/localization/locale.categories/category.time/locale.time.put/locale.time.put.members/put2.pass.cpp

 //===----------------------------------------------------------------------===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.
 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
 //
 //===----------------------------------------------------------------------===//
 
-// XFAIL: LIBCXX-WINDOWS-FIXME
+// FIXME: This XFAIL has no explanation yet.
+// XFAIL: msvc
 
 // <locale>
 
 // class time_put<charT, OutputIterator>
 
 // iter_type put(iter_type s, ios_base& str, char_type fill, const tm* t,
 //               char format, char modifier = 0) const;
 

libcxx/test/std/localization/locales/locale/locale.operators/compare.pass.cpp

 //===----------------------------------------------------------------------===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.
 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
 //
 //===----------------------------------------------------------------------===//
 
 // https://llvm.org/PR41018
-// XFAIL: windows-dll
+// XFAIL: windows-dll && msvc
 
 // <locale>
 
 // template <class charT, class Traits, class Allocator>
 //   bool operator()(const basic_string<charT,Traits,Allocator>& s1,
 //                   const basic_string<charT,Traits,Allocator>& s2) const;
 
 #include <locale>

libcxx/test/std/utilities/function.objects/func.wrap/func.wrap.func/func.wrap.func.con/copy_move.pass.cpp

 //===----------------------------------------------------------------------===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.
 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
 //
 //===----------------------------------------------------------------------===//
 
-// XFAIL: LIBCXX-WINDOWS-FIXME
+// FIXME: This XFAIL has no explanation yet.
+// XFAIL: msvc
 
 // <functional>
 
 // class function<R(ArgTypes...)>
 
 // function(const function&  f);
 // function(function&& f); // noexcept in C++20
 

libcxx/test/std/utilities/function.objects/func.wrap/func.wrap.func/noncopyable_return_type.pass.cpp

 //===----------------------------------------------------------------------===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.
 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
 //
 //===----------------------------------------------------------------------===//
 
 // UNSUPPORTED: c++03, c++11, c++14
 
-// XFAIL: LIBCXX-WINDOWS-FIXME
+// FIXME: This XFAIL has no explanation yet.
+// XFAIL: msvc
 
 // <functional>
 
 #include <functional>
 #include <cassert>
 
 #include "test_macros.h"