33325524f5
The poisoned_hash_helper header was relying on an implicit forward declaration of std::hash located in <type_traits>. When we improve the modularization of the library, that causes issues, in addition to being a fundamentally non-portable assumption in the test suite. It turns out that the reason for relying on a forward declaration is to be able to test that std::hash is *not* provided if we don't include any header that provides it. But testing that is actually both non-portable and not really useful. Indeed, what harm does it make if additional headers provide std::hash specializations? That would certainly be conforming -- the Standard never requires an implementation to avoid providing a declaration when a given header is included, instead it mandates what *must* be provided for sure. In that spirit, it would be conforming for e.g. `<cstddef>` to define the hash specializations if that was our desire. I also don't read https://wg21.link/P0513R0 as going against that statement. Hence, this patch just removes that test which doesn't carry its weight. Fixes #56938
31 lines
787 B
C++
31 lines
787 B
C++
//===----------------------------------------------------------------------===//
|
|
//
|
|
// 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
|
|
|
|
// <system_error>
|
|
|
|
// Test that <system_error> provides all of the arithmetic, enum, and pointer
|
|
// hash specializations.
|
|
|
|
#include <system_error>
|
|
|
|
#include "poisoned_hash_helper.h"
|
|
|
|
#include "test_macros.h"
|
|
|
|
int main(int, char**) {
|
|
test_library_hash_specializations_available();
|
|
{
|
|
test_hash_enabled<std::error_code>();
|
|
test_hash_enabled<std::error_condition>();
|
|
}
|
|
|
|
return 0;
|
|
}
|