[llvm-branch-commits] [libcxx] 39c349e - Add test for spaceship operator to __config

[Hans Wennborg via llvm-branch-commits]

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) &&                                      \