[libcxx] Fix the type attribute for a couple templates

[mstorsjo]

Summary

Use _LIBCPP_TEMPLATE_VIS instead of _LIBCPP_TYPE_VIS for a template class.

This fixes the nodiscard_extensions.pass.cpp and a couple func.search.default test cases when built in MSVC/DLL configurations.

[curdeius]

Could you fix the same thing in experimental/functional as well when you're here?
Also, shouldn't boyer_moore_searcher and boyer_moore_horspool_searcher (in <experimental/functional>) and in_place_index_t (in <utility>) be affected as well?

[mstorsjo]

In D99932#2670597, @curdeius wrote:

Could you fix the same thing in experimental/functional as well when you're here?

Sure

Also, shouldn't boyer_moore_searcher and boyer_moore_horspool_searcher (in <experimental/functional>) and in_place_index_t (in <utility>) be affected as well?

Yes, so it seems. FWIW the experimental library can only be built as a static library, so that's not tested in the current CI setup (but I'm experimenting with extending testing with a static library case as well, which would increase coverage over a few more testcases, and narrow down a number of the existing xfails), but in a static library configuration on windows, both _LIBCPP_TYPE_VIS and _LIBCPP_TEMPLATE_VIS expand to nothing at all, so the distinction isn't quite as easy to pick up on. And I didn't find any test with an XFAIL regarding in_place_index_t, but let's see what another round through the CI says...

[mstorsjo]

Remove XFAILs from more tests, fix the attribute on a couple more classes.

[curdeius]

LGTM. Thanks!

libcxx/include/functional

while (true)
          }
     }
 }
 
 #if _LIBCPP_STD_VER > 14
 
 // default searcher
 template<class _ForwardIterator, class _BinaryPredicate = equal_to<>>
-class _LIBCPP_TYPE_VIS default_searcher {
+class _LIBCPP_TEMPLATE_VIS default_searcher {
 public:
     _LIBCPP_INLINE_VISIBILITY _LIBCPP_CONSTEXPR_AFTER_CXX17
     default_searcher(_ForwardIterator __f, _ForwardIterator __l,
                        _BinaryPredicate __p = _BinaryPredicate())
         : __first_(__f), __last_(__l), __pred_(__p) {}
 
     template <typename _ForwardIterator2>
     _LIBCPP_INLINE_VISIBILITY _LIBCPP_CONSTEXPR_AFTER_CXX17

libcxx/test/libcxx/diagnostics/nodiscard_extensions.pass.cpp

 // This test intentionally leaks memory, so it is unsupported under ASAN.
 // UNSUPPORTED: asan
 
 // AppleClang9 and GCC 5 don't support C++17's implicitly synthesized
 // deduction guides from existing ctors, needed by default_searcher() below.
 // UNSUPPORTED: apple-clang-9
 // UNSUPPORTED: gcc-5
 
-// XFAIL: LIBCXX-WINDOWS-FIXME
-
 // All entities to which libc++ applies [[nodiscard]] as an extension should
 // be tested here and in nodiscard_extensions.fail.cpp. They should also
 // be listed in `UsingLibcxx.rst` in the documentation for the extension.
 
 #include <algorithm>
 #include <functional>
 #include <iterator>
 #include <memory>