[PATCH] D41048: [libcxx] workaround PR 28385 in __find_exactly_one_checked

Marshall Clow via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Dec 10 09:51:56 PST 2017


mclow.lists added inline comments.


================
Comment at: include/tuple:1015
 struct __find_exactly_one_checked {
-  static constexpr bool __matches[] = {is_same<_T1, _Args>::value...};
+#if defined(__clang__) && __clang_major__ > 5 && __cplusplus > 201402L
+    // Workaround https://bugs.llvm.org/show_bug.cgi?id=28385
----------------
We try not to have naked tests like this in the code.

If we have to, maybe:

`#if defined(_LIBCPP_COMPILER_CLANG) && _LIBCPP_CLANG_VER > 500 && _LIBCPP_STD_VER > 14`

but I would rather define a new macro in <__config> and give it a meaningful name.



https://reviews.llvm.org/D41048





More information about the cfe-commits mailing list