Author: David Zarzycki Date: 2020-01-24T19:35:17+01:00 New Revision: 39c349e8fc7f4b334cf4b30724b28dfce44a024e
URL: https://github.com/llvm/llvm-project/commit/39c349e8fc7f4b334cf4b30724b28dfce44a024e DIFF: https://github.com/llvm/llvm-project/commit/39c349e8fc7f4b334cf4b30724b28dfce44a024e.diff LOG: Add test for spaceship operator to __config Summary: The libcxx test suite auto-detects spaceship operator, but __config does not. This means that the libcxx test suite has been broken for over a month when using top-of-tree clang. This also really ought to be fixed before 10.0. See: bc633a42dd409dbeb456263e3388b8caa4680aa0 Reviewers: chandlerc, mclow.lists, EricWF, ldionne, CaseyCarter Reviewed By: EricWF Subscribers: broadwaylamb, hans, dexonsmith, tstellar, llvm-commits, libcxx-commits Tags: #libc, #llvm Differential Revision: https://reviews.llvm.org/D72980 (cherry picked from commit 5dda92fcb0ce9206f831aa7cddf24421dcf044d7) Added: Modified: libcxx/include/__config Removed: ################################################################################ diff --git a/libcxx/include/__config b/libcxx/include/__config index ac4549802ef2..8f48f16c2364 100644 --- a/libcxx/include/__config +++ b/libcxx/include/__config @@ -1350,10 +1350,9 @@ _LIBCPP_FUNC_VIS extern "C" void __sanitizer_annotate_contiguous_container( #define _LIBCPP_HAS_NO_COROUTINES #endif -// FIXME: Correct this macro when either (A) a feature test macro for the -// spaceship operator is provided, or (B) a compiler provides a complete -// implementation. +#if !defined(__cpp_impl_three_way_comparison) || __cpp_impl_three_way_comparison < 201907L #define _LIBCPP_HAS_NO_SPACESHIP_OPERATOR +#endif // Decide whether to use availability macros. #if !defined(_LIBCPP_BUILDING_LIBRARY) && \ _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
