[libcxx][test] Narrow XFAIL for tests that pass with `msvc && stdlib=msvc`

[CaseyCarter]

Summary

... but fail with msvc && stdlib=libc++.

[mstorsjo]

This looks sensible to me. (I would presume that some amount of other windows/msvc specific XFAILs aren't right for stdlib=msvc - I presume you don't run all of the libc++ tests regularly in that configuration, only some subset?)

[CaseyCarter]

In D118194#3270797, @mstorsjo wrote:

This looks sensible to me. (I would presume that some amount of other windows/msvc specific XFAILs aren't right for stdlib=msvc - I presume you don't run all of the libc++ tests regularly in that configuration, only some subset?)

Yes, we run a subset - although I'm trying to get folks motivated to grow that subset. We have an external control mechanism with a list of a few hundred XFAILs and a couple of dozen SKIPPEDs (which are effectively UNSUPPORTED except that we track them differently.)

[mstorsjo]

This was committed in rGb5bada6f85ce6789415d8b9844b6fd3a26d5408f. (@CaseyCarter, the magic keyword is Differential Revision: <url>, not Differential Review: ...)

[CaseyCarter]

In D118194#3273621, @mstorsjo wrote:

This was committed in rGb5bada6f85ce6789415d8b9844b6fd3a26d5408f. (@CaseyCarter, the magic keyword is Differential Revision: <url>, not Differential Review: ...)

<sarcasm>❤️ Phabricator!</sarcasm>

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
 
 // FIXME: When libc++ is linked against vcruntime (i.e. the default config in
 // MSVC mode), the declarations of std::set_new_handler and std::get_new_handler
 // are provided by vcruntime/UCRT's new.h. However, that header actually only
 // declares set_new_handler - it's missing a declaration of get_new_handler.
 
-// XFAIL: msvc
+// XFAIL: msvc && stdlib=libc++
 
 #include <new>
 #include <cassert>
 
 #include "test_macros.h"
 
 void f1() {}
 void f2() {}

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
 //
 //===----------------------------------------------------------------------===//
 
 // FIXME: In MSVC mode, even "std::function<int(int)> f(aref);" causes
 // allocations.
-// XFAIL: msvc
+// XFAIL: msvc && stdlib=libc++
 
 // <functional>
 
 // class function<R(ArgTypes...)>
 
 // function(const function&  f);
 // function(function&& f); // noexcept in C++20